-
Notifications
You must be signed in to change notification settings - Fork 314
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
[ARCTIC-1167][core][hive] fix Trash for restoring deleted files #1223
Conversation
…part of partitions at a time (apache#1220) * support partition ordered by PartitionWeight for OptimizePlan * if not all partitions are optimized, current change snapshot id should set to -1
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #1223 +/- ##
=============================================
- Coverage 52.69% 28.90% -23.80%
- Complexity 523 5158 +4635
=============================================
Files 43 663 +620
Lines 3704 69592 +65888
Branches 354 8021 +7667
=============================================
+ Hits 1952 20114 +18162
- Misses 1622 47542 +45920
- Partials 130 1936 +1806
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 617 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
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.
LGTM.
* cherry pick [ARCTIC-1167][core] Introduce `Trash` for restoring deleted files (#1169) cherry pick [ARCTIC-1016] Moving files will fail if the target parent directory doesn't exist in HDFS #1046 * cherry pick [ARCTIC-1167][core][hive] fix Trash for restoring deleted files (#1223) * add getSyncFileInfoCachePool back for ThreadPool * [ARCTIC-1167][core][hive] fix Unkeyed Table location and metadata file pattern for `Trash` (#1229)
* fix ArcticHadoopFileIO cast error * overwrite file in trash when move * [ARCTIC-1213] Optimizing of Mixed Format Table supports optimizing a part of partitions at a time (#1220) * support partition ordered by PartitionWeight for OptimizePlan * if not all partitions are optimized, current change snapshot id should set to -1 * fix checkstyle * TableTrashManager should extends Serializable
…he#1223) * fix ArcticHadoopFileIO cast error * overwrite file in trash when move * [ARCTIC-1213] Optimizing of Mixed Format Table supports optimizing a part of partitions at a time (apache#1220) * support partition ordered by PartitionWeight for OptimizePlan * if not all partitions are optimized, current change snapshot id should set to -1 * fix checkstyle * TableTrashManager should extends Serializable
Why are the changes needed?
fix #1167, add some hot-fix
Brief change log
How was this patch tested?
Add some test cases that check the changes thoroughly including negative and positive cases if possible
Add screenshots for manual tests if appropriate
Run test locally before making a pull request
Documentation