Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21440
@squito Thank you!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21440
@gatorsmile sure, thats pretty easy. I'll submit a follow up pr.
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21440
cc @squito @mridulm @markhamstra @jerryshao @vanzin @JoshRosen @rxin
@zsxwing
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21440
Although the code quality is pretty good, I am still afraid it could
introduce some unexpected issues. Is that possible we can introduce a conf to
fall back to the previous implementation? We
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93234/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
**[Test build #93234 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93234/testReport)**
for PR 21440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #93234 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93234/testReport)**
for PR 21440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/21440
LGTM, thanks for working on this @squito !
Since there are other ongoing reviews, I will defer to them to merge.
---
-
To
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21440
@mridulm @jerryshao maybe you would be interested in reviewing this as well?
---
-
To unsubscribe, e-mail:
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21440
@tgravescs @felixcheung @zsxwing maybe one of you could take a look? I got
a lgtm from marcelo but he's out for a few weeks, would prefer to get another
approval, plus I'll need review on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92533/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
**[Test build #92533 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92533/testReport)**
for PR 21440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #92533 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92533/testReport)**
for PR 21440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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-unified/625/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
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/21440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92406/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #92406 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92406/testReport)**
for PR 21440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #92406 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92406/testReport)**
for PR 21440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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-unified/530/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
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/21440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92399/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #92399 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92399/testReport)**
for PR 21440 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #92399 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92399/testReport)**
for PR 21440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
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-unified/525/
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21440
@vanzin @JoshRosen this is also ready in the sequence of 2GB limit related
changes.
(I'll update https://github.com/apache/spark/pull/21451 now that the first
change has gone in)
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91259/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
**[Test build #91259 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91259/testReport)**
for PR 21440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/3671/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21440
thanks for the reviews @markhamstra @Ngone51 , I've updated the pr
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #91259 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91259/testReport)**
for PR 21440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91194/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #91194 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91194/testReport)**
for PR 21440 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21440
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/21440
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/3614/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21440
**[Test build #91194 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91194/testReport)**
for PR 21440 at commit
45 matches
Mail list logo