bwzheng2010 commented on code in PR #4738:
URL: https://github.com/apache/hive/pull/4738#discussion_r1335494025
##########
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/ColumnarSplitSizeEstimator.java:
##########
@@ -35,6 +35,9 @@ public class ColumnarSplitSizeEstimator implements
SplitSizeEstimator {
@Override
public long getEstimatedSize(InputSplit inputSplit) throws IOException {
long colProjSize = inputSplit.getLength();
+ if (colProjSize == 0) {
Review Comment:
Hi，thanks for review the code
The reason for judging the split length is 0 at the beginning is to keep the
original logic.
I think，the original logic is that if the columnar projection size or the
inner split has 0 bytes, it does not mean that the length of this split is 0.
Returning Integer.MAX_VALUE is a safer method.
##########
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/ColumnarSplitSizeEstimator.java:
##########
@@ -35,6 +35,9 @@ public class ColumnarSplitSizeEstimator implements
SplitSizeEstimator {
@Override
public long getEstimatedSize(InputSplit inputSplit) throws IOException {
long colProjSize = inputSplit.getLength();
+ if (colProjSize == 0) {
Review Comment:
Hi，thanks for review the code
The reason for judging the split length is 0 at the beginning is to keep the
original logic.
I think，the original logic is that if the columnar projection size or the
inner split has 0 bytes, it does not mean that the length of this split is 0.
Returning Integer.MAX_VALUE is a safer method.
--
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
