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: [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]

Reply via email to