wgtmac opened a new pull request, #1645:
URL: https://github.com/apache/orc/pull/1645

   ### What changes were proposed in this pull request?
   
   This PR aims to fix #1640 by resetting `BooleanRleEncoderImpl::current` and 
`BooleanRleEncoderImpl::bitsRemained` when suppress
   
   
   ### Why are the changes needed?
   
   As #1640 suppress no null present stream leaves dirty data of 
BooleanRleEncoderImpl::current and BooleanRleEncoderImpl::bitsRemained, which 
will be flush to next stripe's present stream if it has some null values.
   
   ### How was this patch tested?
   
   I hava add a test testSuppressPresentStreamInPreStripe, which will construct 
a orc file with two stripe, the first stripe has no null value and seconds 
stripe has some null values. The constructed orc file writer have some dirty 
data in BooleanRleEncoderImpl for present stream. In the test I have add check 
for read ok and read result is same as write.
   
   Closes #1640 .


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