Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21217
@mgaido91 No problem. Please submit the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21217
if it's ok for you @viirya , I am submitting a PR for this then. I'll
specify in the description that the credit should be given to @jamesthomp but
this can be done also by the committer when event
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21217
Thanks @jamesthomp for you work. When we pick this up, I think we can still
give the credit of the work to you.
---
-
To unsubscr
Github user jamesthomp commented on the issue:
https://github.com/apache/spark/pull/21217
I would be glad for @viirya or @mgaido91 to pick this up from me. It sounds
like the remaining work is to move the test from `SQLQuerySuite` into
`AnalysisSuite`, but I'm not certain how to easil
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21217
Thanks for pinging me @HyukjinKwon . I can take it over too, let me know.
Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubsc
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21217
@HyukjinKwon thanks for pinging me. I'd wait for others to take over this
first, if no one does, I can do it later.
---
-
To unsu
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21217
Can anyone take over this then?
cc @kiszk, @mgaido91 and @viirya as well FYI.
---
-
To unsubscribe, e-mail: reviews-
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21217
Ping once more since this can be merged into Spark 3.0, @robert3005 .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spa
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21217
Hi, @robert3005
I know you deleted the branch, but could you try this once more?
At this time, I can review and merge this if this is ready. Also, can we
have a test in `AnalysisSuite`
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93181/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #93181 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93181/testReport)**
for PR 21217 at commit
[`f31cd56`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #93181 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93181/testReport)**
for PR 21217 at commit
[`f31cd56`](https://github.com/apache/spark/commit/f3
Github user jamesthomp commented on the issue:
https://github.com/apache/spark/pull/21217
@HyukjinKwon - I have resolved the conflict with the docs changes. Please
let me know if any additional changes are required.
---
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93059/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #93059 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93059/testReport)**
for PR 21217 at commit
[`6db67e4`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #93059 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93059/testReport)**
for PR 21217 at commit
[`6db67e4`](https://github.com/apache/spark/commit/6d
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21217
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21217
@jamesthomp, mind updating this please?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional com
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90194/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #90194 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90194/testReport)**
for PR 21217 at commit
[`6db67e4`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #90194 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90194/testReport)**
for PR 21217 at commit
[`6db67e4`](https://github.com/apache/spark/commit/6d
Github user jamesthomp commented on the issue:
https://github.com/apache/spark/pull/21217
@gatorsmile - I have added a note to the migration guide.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21217
@jamesthomp Could you document the behavior change in the migration guide?
https://github.com/apache/spark/blame/master/docs/sql-programming-guide.md#L1802
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90138/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #90138 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90138/testReport)**
for PR 21217 at commit
[`662dd2e`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #90138 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90138/testReport)**
for PR 21217 at commit
[`662dd2e`](https://github.com/apache/spark/commit/66
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21217
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90081/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #90081 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90081/testReport)**
for PR 21217 at commit
[`662dd2e`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21217
**[Test build #90081 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90081/testReport)**
for PR 21217 at commit
[`662dd2e`](https://github.com/apache/spark/commit/66
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21217
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21217
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
39 matches
Mail list logo