Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19458
thanks, merging to master/2.2
---
-
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/19458
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/19458
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83048/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19458
**[Test build #83048 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83048/testReport)**
for PR 19458 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19458
**[Test build #83048 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83048/testReport)**
for PR 19458 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19458
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19458
**[Test build #83043 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83043/testReport)**
for PR 19458 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19458
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83043/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19458
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/19458
**[Test build #83043 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83043/testReport)**
for PR 19458 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19458
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user superbobry commented on the issue:
https://github.com/apache/spark/pull/19458
@jerryshao the error looks a bit unrelated.
```
sbt.ForkMain$ForkError: org.scalatest.exceptions.TestFailedException: null
```
The suite passes locally for me.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19458
**[Test build #83034 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83034/testReport)**
for PR 19458 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19458
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/19458
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83034/
Test FAILed.
---
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19458
There's a UT failure
(https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83014/testReport/junit/org.apache.spark.storage/BlockIdSuite/test_bad_deserialization/).
@superbobry
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19458
**[Test build #83034 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83034/testReport)**
for PR 19458 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19458
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/19458
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83014/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19458
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/19458
**[Test build #83014 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83014/testReport)**
for PR 19458 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19458
**[Test build #83014 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83014/testReport)**
for PR 19458 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19458
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19458
ping @superbobry
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user superbobry commented on the issue:
https://github.com/apache/spark/pull/19458
On second thought, the documentation of `DiskBlockManager.getAllFiles`
states that it returns all files in the directories managed by
`DiskBlockManager`. Filtering anything would violate this
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19458
Yes, I agree in any case it should not throw an exception. But in this PR
you filtered out temp shuffle/local blocks, do you think this block is valid or
not, are they blocks?
So I'd
Github user superbobry commented on the issue:
https://github.com/apache/spark/pull/19458
> Instead of filtering out temp blocks, why not adding parsing rule for
`TempLocalBlockId` and `TempShuffleBlockId`?
Because this fixes the issue out 2 out of 3 possible temp files. The
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19458
Instead of filtering out temp blocks, why not adding parsing rule for
`TempLocalBlockId` and `TempShuffleBlockId`? That could also solve the problem.
Since `DiskBlockManager#getAllFiles` doesn't
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19458
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
29 matches
Mail list logo