Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10705#issuecomment-174387270
This has been updated and is now ready for further review. I've gone ahead
and done the reference -> pin renaming, as discussed previously, have fixed the
flaky / failing tests, and have added a bit more documentation.
I'll keep working on trying to complete the remaining checklist items
tonight (adding more debug logging, a debugging feature-flag, etc.) but it
would be great to get feedback + sign-off on the subset of changes + test
changes here, just to make sure there won't be any major new surprise review
comments at the last minute.
---
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]
