kszucs commented on pull request #7805: URL: https://github.com/apache/arrow/pull/7805#issuecomment-661130910
> @kszucs Breaking users is a concern, I'll add an environment variable for both this change and the previous one that can keep the old buggy behavior. Just to clarify: was actually ~5PM Budapest time when you ran these test (i.e. this patch looks like it fixes a bug?) Right, this patch produces the right result. > > I thought the unit test I added for Pandas captured the intent of the ML example? Let me try to run the example by hand in python to see the results. It fixes that as well, just doesn't keep the old buggy behavior. I was considering to just apply the spark patch on the current master to keep the old buggy behavior, but there is still the nested issue. ---------------------------------------------------------------- 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]
