Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/20676
Thanks all , i will close this PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20676
I don't think unroll memory is storage memory from the interface. It's only
true for unified memory manager. I'm -1 on this change unless you can convince
the community to remove the static
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/20676
Hi, @ConeyLiu. I don't think it's about compatibility. Because both
```StaticMemoryManager``` and ```UnifiedMemoryManager``` call the same function
for release unroll memory, which is
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/20676
Yeah, I see that. I'm not sure it's OK to change. But I think we should
follow the interface design, not the underlying implementation.
---
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/20676
In `StaticMemoryManager`, the storage memory and unroll memory is managed
separately, but, unroll memory is also storage memory, so we do not need
release unroll memory really,Just need to
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/20676
This is for compatibility reasons. The memory management also support
legacy memory management (`StaticMemoryManager`). In `StaticMemoryManager`, the
storage memory and unroll memory is managed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87713/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20676
**[Test build #87713 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87713/testReport)**
for PR 20676 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20676
**[Test build #87713 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87713/testReport)**
for PR 20676 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1098/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
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/20676
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/20676
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87704/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20676
**[Test build #87704 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87704/testReport)**
for PR 20676 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20676
**[Test build #87705 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87705/testReport)**
for PR 20676 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87705/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20676
**[Test build #87705 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87705/testReport)**
for PR 20676 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1091/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
**[Test build #87704 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87704/testReport)**
for PR 20676 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1090/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87685/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20676
**[Test build #87685 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87685/testReport)**
for PR 20676 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1073/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
**[Test build #87685 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87685/testReport)**
for PR 20676 at commit
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87669/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
**[Test build #87669 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87669/testReport)**
for PR 20676 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1056/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20676
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/20676
**[Test build #87669 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87669/testReport)**
for PR 20676 at commit
38 matches
Mail list logo