difin commented on code in PR #3606:
URL: https://github.com/apache/hive/pull/3606#discussion_r975377018
##########
ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcSplit.java:
##########
@@ -103,7 +103,7 @@ public OrcSplit(Path path, Object fileId, long offset, long
length, String[] hos
this.isOriginal = isOriginal;
this.hasBase = hasBase;
this.rootDir = rootDir;
- int bucketId = AcidUtils.parseBucketId(path);
+ bucketId = AcidUtils.parseBucketId(path);
Review Comment:
I didn't find explanation in history why bucket id was set in parse method.
Regarding making bucketId final: bucketId is a private class member and cannot
be set outside of OrcSplit. Before my change it was set in parse() method using
as input the getPath() method from its parent class. This method uses 'fs'
class member from the parent class which is package-private and unaccessible
from OrcSplit. So, bucketId is set using path which cannot be modified and it
can have only one value. Therefore in my opinion bucketId can be set final and
it is safe to do. In other words, if bucketId is set in constructor or in parse
method, it can be assigned only same single value which is based on the path
and the path can't be changed after initial assignment in the constructor.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
