wgtmac commented on PR #1028:
URL: https://github.com/apache/parquet-mr/pull/1028#issuecomment-1434018036

   > I don't have a strong opinion on whether to keep or revert the fix. The 
fix won't cause any correctness issue on the engine side because engine will 
filter again.
   
   Same here.
   
   This is not a correctness issue. Rather we should be careful with NULL 
behavior since different engines may have different assumptions. I'd say we 
might lose some optimization with this fix but it is much safer now.


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