ZhangHuiGui commented on issue #2201:
URL: https://github.com/apache/orc/issues/2201#issuecomment-2832908096

   > Yes, the recommended approach is to reuse a single `ColumnVectorBatch` and 
consume it before calling `next`. The lifecycle is bound to the column reader.
   
   Thanks.
   
   But I think this is not reasonable.
   1. From the semantics of the Reader interface:
   When the caller uses `createRowBatch` to create a new `ColumnVectorBatch`, 
and then executes Next to fill the data, the semantics should be that the 
current `ColumnVectorBatch` manages the life cycle of the data itself.
   
   2. From the perspective of ColumnReader implementation, only the 
`StringDictionaryColumnReader` has this problem, and other readers do not have 
this limitation.


-- 
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]

Reply via email to