dongjoon-hyun commented on a change in pull request #568:
URL: https://github.com/apache/orc/pull/568#discussion_r520043679
##########
File path: java/core/src/java/org/apache/orc/impl/ConvertTreeReaderFactory.java
##########
@@ -1396,7 +1396,7 @@ public void nextVector(ColumnVector previousVector,
FilterContext filterContext) throws IOException {
if (inBytesColVector == null) {
// Allocate column vector for file; cast column vector for reader.
- inBytesColVector = new BytesColumnVector();
+ inBytesColVector = new BytesColumnVector(batchSize);
Review comment:
@pgaref . Could you add a test coverage for this change? It seems that
we are missing `Varchar` to `Binary` conversion test case.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
