Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
@tdas
The rationalization of this patch is to group functions which deal with
delta and snapshot files into one so that the difference between delta file and
snapshot file will be clearly
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/21357
While the new code looks a bit cleaner, I am bit hesistant to this complete
rewriting of the entire critical code. We generally do refactor of the code
only if there is some ultimate benefit, e.g.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93278/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #93278 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93278/testReport)**
for PR 21357 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #93278 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93278/testReport)**
for PR 21357 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93109/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #93109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93109/testReport)**
for PR 21357 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #93109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93109/testReport)**
for PR 21357 at commit
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
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/21357
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93070/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #93070 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93070/testReport)**
for PR 21357 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #93070 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93070/testReport)**
for PR 21357 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21357
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
adding cc. to @zsxwing since he has been reviewing PRs for SS so far.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92088/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #92088 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92088/testReport)**
for PR 21357 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #92088 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92088/testReport)**
for PR 21357 at commit
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92083/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
**[Test build #92083 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92083/testReport)**
for PR 21357 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #92083 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92083/testReport)**
for PR 21357 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91686/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #91686 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91686/testReport)**
for PR 21357 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #91686 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91686/testReport)**
for PR 21357 at commit
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
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/21357
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91681/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #91681 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91681/testReport)**
for PR 21357 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #91681 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91681/testReport)**
for PR 21357 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21357
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
Kindly ping again to @tdas
And cc. to @jose-torres @jerryshao @HyukjinKwon @arunmahadevan for
reviewing.
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90754/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
**[Test build #90754 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90754/testReport)**
for PR 21357 at commit
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
cc. @tdas
---
-
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/21357
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/21357
**[Test build #90754 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90754/testReport)**
for PR 21357 at commit
44 matches
Mail list logo