-
Notifications
You must be signed in to change notification settings - Fork 414
Feat: replace sort order #1500
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
Closed
Closed
Feat: replace sort order #1500
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
ccfcbcd
chore: scaffolding
JasperHG90 95da8f3
chore: scaffolding
JasperHG90 253967d
chore: add skeleton for asc/desc methods
JasperHG90 7b5a98e
chore: scaffolding
JasperHG90 304a806
chore: change method names
JasperHG90 48ac5c0
chore: update methods
JasperHG90 a47067c
chore: update methods
JasperHG90 c1ab2ec
chore: update imports
JasperHG90 8f27d14
chore: stupid renames
JasperHG90 d8720f2
chore: lint
JasperHG90 90db60a
chore: docstrings
JasperHG90 1cd2729
test: add integration test for replace sort order
JasperHG90 0a1e781
test: add test for lookup
JasperHG90 a550ccb
refactor: add last sort order id
JasperHG90 8b09255
refactor: add last sort order id and increment
JasperHG90 67b9e52
chore: add imports
JasperHG90 dcaa63f
feat: add apply and commit methods
JasperHG90 ced6a4b
test: remove spec stuff
JasperHG90 43e09a3
chore: remove unused import
JasperHG90 b460c34
chore: add ReplaceSortOrder to the Transaction class
JasperHG90 190071f
chore: lint
JasperHG90 e9475de
Merge branch 'apache:main' into feat/update-sort-order
JasperHG90 eafafaf
Merge branch 'apache:main' into feat/update-sort-order
JasperHG90 ec5f711
chore: renames (replace to update)
JasperHG90 d69a071
chore: renames (replace to update)
JasperHG90 b5a5bd8
test: add test updating sort order
JasperHG90 8080fa5
refactor: remove the sort order builder
JasperHG90 e77a2c1
chore: remove sort order builder
JasperHG90 fc32b28
chore: lint
JasperHG90 fa1aa50
chore: update comment
JasperHG90 1aa6270
Merge branch 'apache:main' into feat/update-sort-order
JasperHG90 2e9cd3f
test: parametrize over iceberg format versions and remove unnused code
JasperHG90 137dbd9
chore: fmt
JasperHG90 d8b9001
Update pyiceberg/table/update/sorting.py
JasperHG90 58d302d
Update pyiceberg/table/__init__.py
JasperHG90 0b543b8
Merge branch 'feat/update-sort-order' of https://github.com/JasperHG9…
JasperHG90 fd0e287
chore: add arg
JasperHG90 5e57697
chore: fmt
JasperHG90 5131903
docs: update docs
JasperHG90 cc1ae1c
chore: set default
JasperHG90 2ae47e3
Merge branch 'apache:main' into feat/update-sort-order
JasperHG90 d99dfdd
chore: lint and update names
JasperHG90 9d77f3f
chore: determine if a sort order is newly added. If so, set the last …
JasperHG90 3f6a953
test: add test for re-using previously defineed sort order
JasperHG90 7223fdd
chore: fmt
JasperHG90 18ced3f
chore: only set last assigned order id when sure that a new sort orde…
JasperHG90 8f425fb
test: add test for reverting back to unsorted sort order
JasperHG90 f9efab3
chore: fmt
JasperHG90 03f0a1b
Merge branch 'apache:main' into feat/update-sort-order
JasperHG90 46fd88b
Merge branch 'apache:main' into feat/update-sort-order
JasperHG90 353178f
Make the CI happy
Fokko File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.