Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22302
Merged to master/branch-2.4.
---
-
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/22302
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95736/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22302
**[Test build #95736 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95736/testReport)**
for PR 22302 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22302
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/22302
**[Test build #95736 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95736/testReport)**
for PR 22302 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22302
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user fjh100456 commented on the issue:
https://github.com/apache/spark/pull/22302
The pr [#20120](https://github.com/apache/spark/pull/20120) was closed,
instead [SPARK-23355](https://issues.apache.org/jira/browse/SPARK-23355) had
resolved it.
Does it mean the same thing
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22302
btw, the behaivour differences of `TABLEPROPERTIES` and `OPTIONS` have
already been documented somewhere?
https://github.com/apache/spark/pull/20120#issuecomment-354704862
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22302
nit. Maybe, it seems that you removed `##`. You may want to simplify the PR
description. This only re-enables the test cases of previous your PR.
```
## What changes were proposed in
Github user fjh100456 commented on the issue:
https://github.com/apache/spark/pull/22302
@dongjoon-hyun @maropu I'm so sorry, and I have changed the description.
Is it ok now ?
---
-
To unsubscribe, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22302
@fjh100456 please do not break the PR format, and can you clean up, too?
---
-
To unsubscribe, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22302
@fjh100456 . I'm @dongjoon-hyun . I think you put a wrong person in the the
PR description. :)
BTW, you don't need to mention the person. It would be enough to mention
SPARK JIRA IDs.
Github user fjh100456 commented on the issue:
https://github.com/apache/spark/pull/22302
@maropu I'd update the PR description, thank you!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22302
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95572/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22302
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/22302
**[Test build #95572 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95572/testReport)**
for PR 22302 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22302
**[Test build #95572 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95572/testReport)**
for PR 22302 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22302
ok to test.
---
-
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/22302
@fjh100456 Can you format the PR description cleanly to make others more
understood?
---
-
To unsubscribe, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22302
@gatorsmile @ueshin Can you trigger this? I checked the related jira and
code and, then I think these tests should be passed in master when
`usingCTAS`=true.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22302
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22302
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22302
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
23 matches
Mail list logo
