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:
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:
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,
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
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
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
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
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/
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
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
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
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
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
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/
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
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
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
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/
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
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
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/
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
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
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
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
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
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
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
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
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
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/
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
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
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/
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
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
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
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
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
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
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/
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
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:
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
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
55 matches
Mail list logo
