BryanCutler commented on pull request #7805:
URL: https://github.com/apache/arrow/pull/7805#issuecomment-661238238


   Just to clarify things, is the main concern with this patch over keeping the 
previous buggy behavior? Besides that are these changes producing correct 
results and passing roundtrip tests?
   It does seem a little late in the game to be making these kind of changes, 
so unless others view this as extremely low risk, I'm in favor of reverting and 
fixing this later when it's not so rushed.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to