Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21657
Thanks! Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93015/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #93015 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93015/testReport)**
for PR 21657 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #93015 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93015/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/961/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93010/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #93010 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93010/testReport)**
for PR 21657 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #93010 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93010/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/956/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21657
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21657
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21657
Please let me do one pass within few days.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21657
@HyukjinKwon @gatorsmile Would you mind to merge the PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
@HyukjinKwon ping
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92834/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92834 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92834/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/832/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #92834 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92834/testReport)**
for PR 21657 at commit
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21657
@HyukjinKwon yes
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21657
@MaxGekk, does this look good to you?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92706/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #92706 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92706/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/747/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #92706 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92706/testReport)**
for PR 21657 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92704/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #92704 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92704/testReport)**
for PR 21657 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92704 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92704/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/745/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92703/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92703 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92703/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
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/744/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92703 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92703/testReport)**
for PR 21657 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
cc: @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92629/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92629 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92629/testReport)**
for PR 21657 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92629 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92629/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/690/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
@HyukjinKwon @MaxGekk plz check?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92590/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #92590 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92590/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/655/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92590 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92590/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
ok, will revive the code here
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21657
Eh .. actually can we revive
https://github.com/apache/spark/commit/64fad0b519cf35b8c0a0dec18dd3df9488a5ed25#diff-d19881aceddcaa5c60620fdcda99b4c4L79
? This sounds safer to me.
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
@HyukjinKwon kindly ping
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
@HyukjinKwon WDYT?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
Not sure though, the `tokenIndexArr` implementation is always faster than
the unsafe projection?
---
-
To unsubscribe, e-mail:
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21657
> Do you mean we remove the option for column pruning in csv?
I mean reverting back the index mapping - `tokenIndexArr`. In this case,
your changes in `buildReader` are not needed.
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
@MaxGekk Do you mean we remove the option for column pruning?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21657
Actually the issue happened because I removed the mapping:
https://github.com/apache/spark/commit/64fad0b519cf35b8c0a0dec18dd3df9488a5ed25#diff-d19881aceddcaa5c60620fdcda99b4c4L79
I would
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92445/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #92445 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92445/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/559/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
**[Test build #92445 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92445/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/21657
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92418/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92418 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92418/testReport)**
for PR 21657 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21657
**[Test build #92418 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92418/testReport)**
for PR 21657 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
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/542/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21657
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21657
@HyukjinKwon @MaxGekk
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
81 matches
Mail list logo
