Title: Remove feature set update from job update cycle by zhilingc · Pull Request #711 · feast-dev/feast · GitHub
Open Graph Title: Remove feature set update from job update cycle by zhilingc · Pull Request #711 · feast-dev/feast
X Title: Remove feature set update from job update cycle by zhilingc · Pull Request #711 · feast-dev/feast
Description: What this PR does / why we need it: Removes the feature set update from the job update cycle, because job updates are long running and are write-transactional, and so can block feast core for long periods of time. Hopefully this improves the performance of the system. The responsibility of making feature set status updates is pushed to when the feature set is retrieved (either by ListFeatureSets or GetFeatureSet). Would like the dataflow tests to be fixed and run on this PR before this is merged in, ideally. Which issue(s) this PR fixes: Fixes #664 Does this PR introduce a user-facing change?:
Open Graph Description: What this PR does / why we need it: Removes the feature set update from the job update cycle, because job updates are long running and are write-transactional, and so can block feast core for long ...
X Description: What this PR does / why we need it: Removes the feature set update from the job update cycle, because job updates are long running and are write-transactional, and so can block feast core for long ...
Opengraph URL: https://github.com/feast-dev/feast/pull/711
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/checks(.:format) |
| route-controller | pull_requests |
| route-action | checks |
| fetch-nonce | v2:272d76dc-c773-2edd-d279-2ab2e302da3a |
| current-catalog-service-hash | 87dc3bc62d9b466312751bfd5f889726f4f1337bdff4e8be7da7c93d6c00a25a |
| request-id | C526:A5EEF:C9911A:103A02C:697F87E8 |
| html-safe-nonce | 2b257b2f90632bf97b8ccd675b1ec6966822c4e62640c527510c84f466a43c37 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNTI2OkE1RUVGOkM5OTExQToxMDNBMDJDOjY5N0Y4N0U4IiwidmlzaXRvcl9pZCI6IjMyNzAwNjYwMDM2MTQwNzQ4NTYiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | c28bedf82d8222c5cbe64baeb5d8f97269646661c7690541d7f44ef71616a9d9 |
| hovercard-subject-tag | pull_request:418518076 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,checks,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/feast-dev/feast/pull/711/checks |
| twitter:image | https://avatars.githubusercontent.com/u/15104168?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/15104168?s=400&v=4 |
| og:image:alt | What this PR does / why we need it: Removes the feature set update from the job update cycle, because job updates are long running and are write-transactional, and so can block feast core for long ... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 60279d4097367e16897439d16d6bbe4180663db828c666eeed2656988ffe59f6 |
| turbo-cache-control | no-preview |
| go-import | github.com/feast-dev/feast git https://github.com/feast-dev/feast.git |
| octolytics-dimension-user_id | 57027613 |
| octolytics-dimension-user_login | feast-dev |
| octolytics-dimension-repository_id | 161133770 |
| octolytics-dimension-repository_nwo | feast-dev/feast |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 161133770 |
| octolytics-dimension-repository_network_root_nwo | feast-dev/feast |
| turbo-body-classes | logged-out env-production page-responsive full-width full-width-p-0 |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 7c85641c598ad130c74f7bcc27f58575cac69551 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width