-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[Python] support partition spec in iceberg python library #3228
Comments
@jun-he shall I work on this? |
This depends on the work in #3216 for types definition. |
Yep, working on #3216 and will ship it ASAP. |
@jun-he reason for closing this? |
Sorry, that was not intended. Reopened it. |
Thanks @jun-he. I should have a PR ready sometime next week. |
Now that schema has been merged I think we need a couple more PRs in order to close this:
For 3. I know we discussed whether we should go with the builder pattern or just lean into python's ability for named optional arguments. There might be a middle ground in there where we use the builder pattern (to maintain consistency with the java codebase) and simply use that builder in the Anyone interested, please feel free to tackle any of these items! 😃 |
I can help by starting with PartitionSpec! |
This was added in the PRs above, so I'll close this. |
No description provided.
The text was updated successfully, but these errors were encountered: