Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22952
@zsxwing @dongjoon-hyun @steveloughran
Thanks all for the valuable feedback! I applied review comments.
While I covered the new feature with new UTs, I'm yet to test this manually
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22952
**[Test build #98919 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98919/testReport)**
for PR 22952 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22952
**[Test build #98918 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98918/testReport)**
for PR 22952 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22952
**[Test build #98917 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98917/testReport)**
for PR 22952 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/22952
> Provide additional option: delete (two options - 'rename' / 'delete' -
are mutually exclusive)
>
> Actually the actions end users are expected to take are 1. moving to
archive directory
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22952
cc. @zsxwing
---
-
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/22952
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98493/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22952
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/22952
**[Test build #98493 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98493/testReport)**
for PR 22952 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22952
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/22952
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98491/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22952
**[Test build #98491 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98491/testReport)**
for PR 22952 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22952
**[Test build #98493 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98493/testReport)**
for PR 22952 at commit
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22952
I feel the patch is simple to skip verifying manually against HDFS, but
I'll try to spin up HDFS cluster and test this manually.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22952
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/22952
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22952
**[Test build #98491 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98491/testReport)**
for PR 22952 at commit
17 matches
Mail list logo