-
Notifications
You must be signed in to change notification settings - Fork 905
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
Support versioning of the underlying dataset with PartitionedDataset #521
Conversation
5ed19e2
to
61dba32
Compare
Co-authored-by: Dmitrii Deriabin <44967953+DmitriiDeriabinQB@users.noreply.github.com>
611235d
to
91ecc39
Compare
b2228df
to
cc50803
Compare
a67c808
to
e34f58b
Compare
@DmitriiDeriabinQB Changes made/test issues resolved; this PR should be good to go! 🤞 |
Re-open to be part of research. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The docs change (fixes the broken link in Prefect docs) LGTM. I cannot sign off on the others 🤷 Thanks @deepyaman 🏆 🏅 🌟
Should we close this now that kedro-org/kedro-plugins#447 was opened? |
Doesn't matter to me; I'd set this to be closed automatically by kedro-org/kedro-plugins#447 already, anyway. |
Closing in favour of kedro-org/kedro-plugins#447 |
Description
Development notes
Checklist
RELEASE.md
fileNotice
I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":
I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.
I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.