Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/11939#issuecomment-208533723
Would be good to update to master so can run the latest integration tests,
but at its current point it seems to have gotten all of the direct paramMap
sets (although there may be more in master now). It might make sense to also
add a clearParam function and then add a note that no one (including
developers) should directly access the param map but instead use one of the
access functions?
---
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]
