tgravescs commented on pull request #29869: URL: https://github.com/apache/spark/pull/29869#issuecomment-701548103
thanks for the explanation but that isn't the intention of that field. Not making things worse is great but doesn't indicate whether it is user facing. I can add a completely new api to RDD that if not used doesn't "break" anything but its adding a user facing API and extra functionality that needs to be documented so the user knows its there and can use it. this "api" change is meant for end users to use, so how do they know its there to use - most users don't go read the code to find little options like this. this is adding a somewhat obscure special prefix which does special things. I'm not sure I really like that approach. The nice thing about it is it doesn't require any function API changes, but that also makes it not as obvious. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
