[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-19 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/22413 I see. Thank you, @MaxGekk ! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-19 Thread MaxGekk
Github user MaxGekk commented on the issue: https://github.com/apache/spark/pull/22413 @dongjoon-hyun It cannot be merged to 2.3 easily because `DataSourceV2Relation` doesn't have the `options` field, and the test for read is not compilable. I will try to fix it tomorrow. ---

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-19 Thread MaxGekk
Github user MaxGekk commented on the issue: https://github.com/apache/spark/pull/22413 @dongjoon-hyun Here is the PR for 2.4: https://github.com/apache/spark/pull/22474 . I will check does it have conflicts in 2.3 and if so, I will backport it to 2.3 ---

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-19 Thread rdblue
Github user rdblue commented on the issue: https://github.com/apache/spark/pull/22413 Thanks @MaxGekk, sorry for the original omission! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-19 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/22413 Thank you so much, @MaxGekk . It would be great if we can have that Spark 2.4.0 RC2. --- - To unsubscribe, e-mail:

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-19 Thread MaxGekk
Github user MaxGekk commented on the issue: https://github.com/apache/spark/pull/22413 > There is some conflict. Please make backporting PRs to 2.4 and 2.3. > BTW, @MaxGekk . Could you send a backport PR to branch-2.4? Sure, I will do that. ---

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-19 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/22413 Oh, it was @MaxGekk . I helped some test cases. BTW, @MaxGekk . Could you send a backport PR to `branch-2.4`? --- - To

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-19 Thread rdblue
Github user rdblue commented on the issue: https://github.com/apache/spark/pull/22413 +1, thanks for fixing this, @dongjoon-hyun! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-15 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/22413 Thank you, @MaxGekk . Merged to master/2.4/2.3. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22413 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96098/ Test PASSed. ---

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22413 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22413 **[Test build #96098 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96098/testReport)** for PR 22413 at commit

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22413 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22413 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96096/ Test PASSed. ---

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22413 **[Test build #96096 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96096/testReport)** for PR 22413 at commit

[GitHub] spark issue #22413: [SPARK-25425][SQL] Extra options should override session...

2018-09-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22413 **[Test build #96098 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96098/testReport)** for PR 22413 at commit