okumin commented on code in PR #5409:
URL: https://github.com/apache/hive/pull/5409#discussion_r1824723103
##########
ql/src/java/org/apache/hadoop/hive/ql/io/HiveInputFormat.java:
##########
@@ -171,6 +173,19 @@ public String inputFormatClassName() {
return inputFormatClassName;
}
+ public OptionalInt getBucketId() {
+ if (inputSplit instanceof PartitionAwareSplit) {
+ final PartitionAwareSplit split = (PartitionAwareSplit) inputSplit;
Review Comment:
It seems to be a valid request. The original APIs are the result of
considering future flexibility especially when we unlock this feature to
non-bucketed partition transforms that might not have # of buckets. Anyway, the
interface is marked as unstable, and we don't know the best API for them now
https://github.com/apache/hive/pull/5409/commits/aec8bd2a425d1886c07455d260ff3142042e9efa
--
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: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
