[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/22940 Hm .. okay. let me close this for now. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/22940 Hmmm yea but like .. some of classes similar with this case have been renamed time to time, for instancem `json InferSchema` -> `json JSONInferSchema` when CSV datasource was added

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/22940 hmhm .. it's trivial and yea it is a logical change. I happened to take a look some codes around here lately, and the name `SQLUtils` actually annoyed me few times :(. I will leave it to

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user srowen commented on the issue: https://github.com/apache/spark/pull/22940 I'm neutral on it... seems like a logical change but is there any issue (like ambiguous imports that are annoying) other than not matching the pattern? I am not super concerned about back-porting

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22940 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98443/ Test PASSed. ---

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22940 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22940 **[Test build #98443 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98443/testReport)** for PR 22940 at commit

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22940 **[Test build #98443 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98443/testReport)** for PR 22940 at commit

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22940 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22940 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22940: [MINOR][R] Rename SQLUtils name to RSQLUtils in R

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/22940 @felixcheung, I don't feel super strongly about it but it bugged me. WDYT? --- - To unsubscribe, e-mail: