-
Notifications
You must be signed in to change notification settings - Fork 998
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Backports for v0.6.2 #918
Backports for v0.6.2 #918
Conversation
* Clean commit of dependency changes for spring-boot 2.3.0 * Bump to 2.3.1 * Remove exlusions and Javadoc config * Remove logging exclusion for ingestion * Increase versions of grpc, protoc, protobuf, hibernate, and grpc-starter * Bump version of kafka-clients and spring-security-oauth * Upgrade opencensus, beam, and google-cloud versions * fix ingestion tests * runtime deps * ingestion spec update: should take some time before ack Co-authored-by: Christopher Wirick <crwirick@gmail.com> Co-authored-by: Willem Pienaar <git@willem.co>
* Cache authorization. * fix formatting, removed default key generator and added a bean. * fix rebase errors.
* Authentication and authorization for feast serving, squashed on 07/21 * fix e2e, add metadata plugin in jobs, merge labels, auth failure test, removed unwanted expire time validation from gauth. * fix rebase adaption. * Fix core integration test. * Authentication integration test. * Add authorization test and minor refactoring. * fix failing integration test. * fix lint error.
* Fix ingestion with same featureset name * Address PR comments * Remove unrelated test * Remove unnecessary line Co-authored-by: Terence <terence.limxp@go-jek.com> Co-authored-by: Willem Pienaar <6728866+woop@users.noreply.github.com>
…update. (#908) * Update RedisCustomIO to write FeatureRows with field's name set to hash of field. * Update FeatureRowDecoder to decode by name hash instead of order * Bump pytest order numbers by 2 to make space for new tests * Revert "Bump pytest order numbers by 2 to make space for new tests" This reverts commit aecc9a6e9a70be3fd84d04f81442b518be01a4c6. * Added e2e to check that feature rows with missing or extra fields can be retrieved * Clarify docs about Feature Row v1 encoding and Feature Row v2 encoding * Fix python lint * Update FeatureRowDecoder's isEncodedV2 check to use anyMatch() * Make missing field/extra field e2e tests independent of other tests. * Update FeatureRowDecoder if/else statement into 2 ifs * Fix python and java lint * Fix java unit test failures * Fix ImportJobTest java unit test * Sync github workflows with master * Sync .github folder with master for fix * Replace v1/v2 encoding with v1/v2 decoder in docs
…nds is missing (#917) * add checkArgument * add default write_triggering_frequency_seconds
* diskType & streamingEnginer * edit infra docs
* redis sink read then write * fix load tests * e2e * fix * specify feature ref * move test up in order * set project default * some docs * reorder e2e tests * reorder e2e tests * reorder e2e tests
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: mrzzy The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test test-end-to-end |
/test test-end-to-end-batch |
@mrzzy: The following test failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
What this PR does / why we need it
Backports PRs to be released in version 0.6.2:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: