alexeykudinkin commented on code in PR #7402:
URL: https://github.com/apache/hudi/pull/7402#discussion_r1044795993
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/execution/bulkinsert/BulkInsertSortMode.java:
##########
@@ -22,7 +22,8 @@
* Bulk insert sort mode.
*/
public enum BulkInsertSortMode {
- NONE,
- GLOBAL_SORT,
- PARTITION_SORT
+ NONE,
+ GLOBAL_SORT,
+ PARTITION_SORT,
+ PARTITION_PATH_REDISTRIBUTE
Review Comment:
> not for non-partitioned tables, so I don't want to give the impression
that the repartition is also done for non-partitioned tables.
I don't think there will be such confusion -- it's prefixed w/
`PARTITION_COLUMN` which doesn't exist for non-partitioned tables.
See, `PARTITION_PATH_SORT` leaves an impression that we sort by partition
path, which is not what we're trying to do here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
