Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20700
a late LGTM :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/20700
I am having some problems with the merge script, give me a little bit of
time.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/20700
Merging to master. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20700
Good catch. I removed the old statement from the description.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/20700
@kiszk can you update the PR description?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20700
@rednaxelafx thanks, I integrated some of your changes into PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87899/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
**[Test build #87899 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87899/testReport)**
for PR 20700 at commit
[`4069e1f`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1237/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
**[Test build #87899 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87899/testReport)**
for PR 20700 at commit
[`4069e1f`](https://github.com/apache/spark/commit/40
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Merged build finished. Test FAILed.
---
-
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/20700
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87897/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20700
**[Test build #87897 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87897/testReport)**
for PR 20700 at commit
[`3837eff`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1236/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20700
**[Test build #87897 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87897/testReport)**
for PR 20700 at commit
[`3837eff`](https://github.com/apache/spark/commit/38
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20700
**[Test build #87894 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87894/testReport)**
for PR 20700 at commit
[`4538ebc`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Merged build finished. Test FAILed.
---
-
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/20700
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87894/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1233/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
**[Test build #87894 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87894/testReport)**
for PR 20700 at commit
[`4538ebc`](https://github.com/apache/spark/commit/45
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1229/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
**[Test build #87889 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87889/testReport)**
for PR 20700 at commit
[`7da3e6b`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Merged build finished. Test FAILed.
---
-
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/20700
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87889/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20700
**[Test build #87889 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87889/testReport)**
for PR 20700 at commit
[`7da3e6b`](https://github.com/apache/spark/commit/7d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Merged build finished. Test FAILed.
---
-
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/20700
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87887/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20700
**[Test build #87887 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87887/testReport)**
for PR 20700 at commit
[`ebce1f2`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
**[Test build #87886 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87886/testReport)**
for PR 20700 at commit
[`f589a2a`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87886/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20700
**[Test build #87887 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87887/testReport)**
for PR 20700 at commit
[`ebce1f2`](https://github.com/apache/spark/commit/eb
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1228/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
**[Test build #87886 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87886/testReport)**
for PR 20700 at commit
[`f589a2a`](https://github.com/apache/spark/commit/f5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1227/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20700
@rednaxelafx Oh, very interesting since we are doing the similar thing in
West coast and Japan!
I just say `not refactored YET`. Yeah, I absolutely love to delete old
instance method if we to
Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/20700
Aha! Thanks @kiszk san for working on this! I really wanted the stateless
methods to be extracted so that I can use more utils without having to pass
around a `CodegenContext` for no good.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Merged build finished. Test FAILed.
---
-
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/20700
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87881/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20700
**[Test build #87881 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87881/testReport)**
for PR 20700 at commit
[`9fed753`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20700
**[Test build #87881 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87881/testReport)**
for PR 20700 at commit
[`9fed753`](https://github.com/apache/spark/commit/9f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1224/
Tes
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20700
@mgaido91 thanks. Let me ping @cloud-fan and @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additi
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20700
I like this approach. Let's see others' opinion, but it seems a good thing
to do up to me.
---
-
To unsubscribe, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87837/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20700
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/20700
**[Test build #87837 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87837/testReport)**
for PR 20700 at commit
[`f8b478f`](https://github.com/apache/spark/commit/f
55 matches
Mail list logo