Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93327/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #93327 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93327/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #93327 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93327/testReport)**
for PR 20057 at commit
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Just made the changes you mentioned @gatorsmile :-)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93322/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #93322 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93322/testReport)**
for PR 20057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #93322 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93322/testReport)**
for PR 20057 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92963/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #92963 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92963/testReport)**
for PR 20057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #92963 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92963/testReport)**
for PR 20057 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
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/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92841/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #92841 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92841/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #92841 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92841/testReport)**
for PR 20057 at commit
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@dongjoon-hyun @gatorsmile Any update, guys?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@gatorsmile @dongjoon-hyun Guys, any update?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/20057
Gentle ping @gatorsmile @dongjoon-hyun ð
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Gentle ping again @gatorsmile @dongjoon-hyun :-)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Gentle ping @gatorsmile :-)
---
-
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/20057
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88057/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #88057 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88057/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #88057 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88057/testReport)**
for PR 20057 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20057
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20057
The failures are
- Last time, `RateSourceV2Suite`.
- This time, `HiveClientSuites`.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88052/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #88052 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88052/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #88052 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88052/testReport)**
for PR 20057 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
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/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88040/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #88040 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88040/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #88040 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88040/testReport)**
for PR 20057 at commit
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@dongjoon-hyun I saw that Spark 2.3 was released a few days ago, congrats
on the release! :-) Is there anything stopping us from merging this PR into
master now?
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87591/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #87591 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87591/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87591 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87591/testReport)**
for PR 20057 at commit
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@dongjoon-hyun Made the changes you pointed out, thanks! ð
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87582/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #87582 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87582/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87582 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87582/testReport)**
for PR 20057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87543/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #87543 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87543/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87543 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87543/testReport)**
for PR 20057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87531/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #87531 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87531/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87531 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87531/testReport)**
for PR 20057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #87530 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87530/testReport)**
for PR 20057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87530/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87530 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87530/testReport)**
for PR 20057 at commit
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Thanks again @dongjoon-hyun, much appreciated ð. I've made changes
accordingly, and added comments in this PR where appropriate.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87529 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87529/testReport)**
for PR 20057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87529/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
**[Test build #87529 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87529/testReport)**
for PR 20057 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20057
I finished my second round. Could you update once more?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87524/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87524 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87524/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87524 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87524/testReport)**
for PR 20057 at commit
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Thanks for your review @dongjoon-hyun ð I've corrected all the
indentation problems (didn't show up locally when running scalastyle checks for
some reason). Added comments where necessary
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20057
Hi, @danielvdende. I finished my first round review.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Thanks guys! @gatorsmile @dongjoon-hyun
Happy to help out expanding Spark SQL JDBC where necessary to match and
improve on sqoop :-)
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20057
Thank you for pinging me, @gatorsmile . Yep. I'll take a look both
`OrcQuerySuite` and this PR in this morning. Sorry for late response,
@danielvdende .
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20057
This test is a flaky test. Your changes did not fail any test case. I will
review your PR after the 2.3 release. Thanks again!
cc @dongjoon-hyun Do you want to take a look at this?
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20057
Our overwrite semantics is confusing to most. We need to correct it in the
next release, i.e., Spark 2.4.
Even if we try our best to keep the schema of the original table, the
actual
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Hi guys, @Fokko @gatorsmile, completely agree with what @Fokko mentioned,
our main reason for wanting to get away from Sqoop is also for stability
reasons and to get rid of MapReduce in
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Hmm, not it fails the OrcQuerySuite. This PR doesn't touch any of the Orc
implementation in Spark. Could this be a flaky test @gatorsmile ?
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/20057
Hi @gatorsmile, thanks for putting it to the tests. The main reason why I
personally dislike Sqoop is:
- **Legacy.** The old map-reduce should be buried in the upcoming years. As
a data
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87493/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87493 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87493/testReport)**
for PR 20057 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87493 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87493/testReport)**
for PR 20057 at commit
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
Tests are failing on a spark streaming test. I think it's probably because
of the age of this PR, will rebase to get the changes into the PR that were
merged into master since I opened the PR
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87484 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87484/testReport)**
for PR 20057 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87484/
Test FAILed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20057
@danielvdende @Fokko We definitely want to help the community replace Sqoop
by Spark SQL. However, `truncate` is only used when users use
SaveMode.Overwrite to write the external JDBC tables. In
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20057
**[Test build #87484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87484/testReport)**
for PR 20057 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20057
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/20057
We're in the process of integrating Spark in Airflow, and support for the
`cascadeTruncate` is required to make this succeed. First steps are here:
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@dongjoon-hyun @gatorsmile sorry to keep asking, but could you let me know
when we can get this merged?
---
-
To
Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/20057
Any idea when this will be merged into master? We could use this since we
are ditching sqoop ð
---
-
To unsubscribe, e-mail:
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@Stephan202 thanks for pointing out those docs issues, just pushed the
changes :-).
@gatorsmile @dongjoon-hyun would you have a chance to take a look at this
again?
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20057
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/20057
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user danielvdende commented on the issue:
https://github.com/apache/spark/pull/20057
@dongjoon-hyun @gatorsmile any further update?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
1 - 100 of 108 matches
Mail list logo
