Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14678
Thanks but also FYI this is not "minor". We should have a JIRA ticket for
changes like this in the future.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14678
thanks, merging to master!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14678
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65263/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14678
**[Test build #65263 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65263/consoleFull)**
for PR 14678 at commit
[`a26c08e`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14678
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14678
**[Test build #65263 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65263/consoleFull)**
for PR 14678 at commit
[`a26c08e`](https://github.com/apache/spark/commit/a
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14678
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14678
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the fea
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14678
@cloud-fan Do you mind if I ask to take a look please?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not ha
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14678
ping @rxin
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or i
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14678
(@rxin cc you in case you just missed this)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fea
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14678
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14678
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63959/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14678
**[Test build #63959 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63959/consoleFull)**
for PR 14678 at commit
[`a26c08e`](https://github.com/apache/spark/commit/
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14678
LGTM pending tests
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes s
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14678
**[Test build #63959 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63959/consoleFull)**
for PR 14678 at commit
[`a26c08e`](https://github.com/apache/spark/commit/a
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14678
@gatorsmile Thanks for a very close look. I should've check the description
of `PARALLEL_PARTITION_DISCOVERY_THRESHOLD`.
---
If your project is set up for it, you can reply to this email and ha
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14678
I checked the conf changes and the data types. It looks good to me. Only
leave a minor comment on the conf description of one related parameter
`PARALLEL_PARTITION_DISCOVERY_THRESHOLD`. Thanks!
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14678
nvm, I got the answer. We are unable to change them. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does n
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14678
Just searched the code and found more. Any reason why we do not change
these?
```
ParquetFileFormat.scala:85:if
(conf.get(SQLConf.PARQUET_OUTPUT_COMMITTER_CLASS.key) == null) {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14678
Sure, will review it today. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enab
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14678
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14678
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63899/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14678
**[Test build #63899 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63899/consoleFull)**
for PR 14678 at commit
[`a57dd5e`](https://github.com/apache/spark/commit/
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14678
Looks good, but I didn't look super carefully.
@gatorsmile do you have time to take a more careful look at this?
---
If your project is set up for it, you can reply to this email and have yo
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14678
cc @rxin, Could you check if this make sense please?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14678
**[Test build #63899 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63899/consoleFull)**
for PR 14678 at commit
[`a57dd5e`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14678
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14678
**[Test build #63898 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63898/consoleFull)**
for PR 14678 at commit
[`c959f3b`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14678
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63898/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14678
**[Test build #63898 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63898/consoleFull)**
for PR 14678 at commit
[`c959f3b`](https://github.com/apache/spark/commit/c
31 matches
Mail list logo
