HyukjinKwon commented on code in PR #42356:
URL: https://github.com/apache/spark/pull/42356#discussion_r1285288659
##########
R/pkg/NAMESPACE:
##########
@@ -485,7 +485,6 @@ export("as.DataFrame",
"cacheTable",
"clearCache",
"createDataFrame",
- "createExternalTable",
Review Comment:
Oh I apologize that I forgot we already discussed that the next Spark
version would likely be 4.0.0 - my bad.
My concern is mainly about "Considerations when breaking APIs" at
https://spark.apache.org/versioning-policy.html. We discussed about removing
some of deprecated API including `SQLContext` at Spark 3.0.0 but we did not
remove in the end because it might be too breaking changes. At that time, we
had to remove and revert some of those PRs (see also
https://github.com/apache/spark/pull/22843 and
https://github.com/apache/spark/pull/27815).
I personally would like to have a bit of discussion in the mailing list
(probably after 3.5.0 release) before we remove some, and find out that some of
them are too much breaking changes.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
