dongjoon-hyun commented on pull request #31571: URL: https://github.com/apache/spark/pull/31571#issuecomment-780179070
No, I don't. `Removing a config w/o deprecation` sounds wrong. > Do you consider a situation when you can just remove this config w/o deprecation only because it is marked as .internal()? No. I disagree with the expression, `de facto`. The documentation change is far much easier than removing `non-internal` config. > it is external de facto. Like https://github.com/databricks/spark-avro says, only `Databricks customer` can use this library directly . All other distributions, especially 3.2.0+ versions, I don't think this PR makes sense. > Databricks customers can also use this library directly on the Databricks Unified Analytics Platform without any additional dependency configurations. ---------------------------------------------------------------- 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]
