Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14399
This looks good - I might add a comment about why we are using this
approach instead of listOfElements in the code so that whoever comes along next
understands why we couldn't use it.
Also just to make sure - when you used listOfElements in your previous
attempt the test suite exposed the issue? If not we should add a test for this
using an old format file (looking at ParqueTest it seems like it should already
be caught I just want to make sure)
Thanks so much for doing this! :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
