Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14567
Hi @Stibbons maybe you can break down a bit more what the relationship
between your two PRs and update the descriptions? I'm maybe a getting a bit
fuzzy on what is intended to go where.
I like splitting up the PRs especially when they become this big - but I'm
thinking if your going to do a huge re-org in the next PR it would make sense
to have that PR enable full tests for that directory - perhaps as part of
lint-python or similar mechanism so that as we incrementally fix sub-components
we can ensure we don't move backwards.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
