Open HashMethod to all types #1171
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
eapolinario
requested review from
wild-endeavor,
kumare3 and
pingsutw
as code owners
Signed-off-by: Eduardo Apolinario <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #1171 +/- ##
==========================================
+ Coverage 68.51% 68.53% +0.02%
==========================================
Files 288 288
Lines 26085 26191 +106
Branches 2918 2925 +7
==========================================
+ Hits 17871 17951 +80
- Misses 7735 7762 +27
+ Partials 479 478 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
wild-endeavor
approved these changes
Sep 23, 2022
Ln11211
pushed a commit
to Ln11211/flytekit
that referenced
this pull request
Oct 2, 2022
* Open HashMethod to all types * Fix test in test_type_hints.py Signed-off-by: Eduardo Apolinario <[email protected]> Signed-off-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]> Signed-off-by: LN <[email protected]>
eapolinario
added a commit
that referenced
this pull request
Oct 7, 2022
…mote.sync (#1212) * Open HashMethod to all types (#1171) * Open HashMethod to all types * Fix test in test_type_hints.py Signed-off-by: Eduardo Apolinario <[email protected]> Signed-off-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]> Signed-off-by: LN <[email protected]> * Return None for SyncCheckpoint.read() when src is empty (#1189) Signed-off-by: Andrew Dye <[email protected]> Signed-off-by: Andrew Dye <[email protected]> Signed-off-by: LN <[email protected]> * fix: plugins/flytekit-k8s-pod/requirements.txt to reduce vulnerabilities (#1192) The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-PROTOBUF-3031740 Signed-off-by: LN <[email protected]> * pyflyte non-fast register (#1205) * pyflyte run non-fast register Signed-off-by: Kevin Su <[email protected]> * lint Signed-off-by: Kevin Su <[email protected]> Signed-off-by: Kevin Su <[email protected]> Signed-off-by: LN <[email protected]> * Update control_plane.rst Added about large response while using Remote.Sync() and how to handle it in the doc Signed-off-by: LN <[email protected]> * Update docs/source/design/control_plane.rst updating FlyteRemote sync message Co-authored-by: Samhita Alla <[email protected]> * Update control_plane.rst Updated the fix for the error and also removed the repeated error message line. Signed-off-by: LN <[email protected]> Signed-off-by: Eduardo Apolinario <[email protected]> Signed-off-by: LN <[email protected]> Signed-off-by: Andrew Dye <[email protected]> Signed-off-by: Kevin Su <[email protected]> Signed-off-by: LN <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]> Co-authored-by: Andrew Dye <[email protected]> Co-authored-by: Snyk bot <[email protected]> Co-authored-by: Kevin Su <[email protected]> Co-authored-by: Samhita Alla <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
TL;DR
Ensure all types can have their hash values overriden
Type
Are all requirements met?
Complete description
Initially we made it so that only pandas dataframes and StructuredDatasets could leverage the ability to override their hash values for the purpose of producing cache keys. This PR lifts that restriction and opens up that feature to all types.
Tracking Issue
flyteorg/flyte#2912
Follow-up issue
NA
OR
https://github.com/flyteorg/flyte/issues/