[GitHub] [spark] srowen commented on a diff in pull request #42356: [SPARK-44685][SQL] Remove deprecated Catalog#createExternalTable

["srowen (via GitHub)" <gi...@apache.org>]

srowen commented on code in PR #42356:
URL: https://github.com/apache/spark/pull/42356#discussion_r1285282846


##########
R/pkg/NAMESPACE:
##########
@@ -485,7 +485,6 @@ export("as.DataFrame",
        "cacheTable",
        "clearCache",
        "createDataFrame",
-       "createExternalTable",

Review Comment:
   Certainly, there may be an argument to not remove some methods. But these can only be removed at major versions; removing one does not imply all deprecated methods must be removed, either. What do you mean by 'going for 4.0.0' and how does this relate to SQLContext? there is indeed not one single plan for all deprecated methods, but what is the argument against this particular one?



-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org