Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-11583] [Core]MapStatus Using RoaringBitmap More Properly #9746

Closed
wants to merge 11 commits into from

Conversation

davies
Copy link
Contributor

@davies davies commented Nov 16, 2015

This PR upgrade the version of RoaringBitmap to 0.5.10, to optimize the memory layout, will be much smaller when most of blocks are empty.

This PR is based on #9661 (fix conflicts), see all of the comments at #9661 .

@SparkQA
Copy link

SparkQA commented Nov 16, 2015

Test build #2066 has finished for PR 9746 at commit 1716a24.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):\n * class BitSet(numBits: Int) extends Serializable\n * class StreamingListener(object):\n

Conflicts:
	core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala
@davies
Copy link
Contributor Author

davies commented Nov 16, 2015

Include #9748 here to pass tests

@SparkQA
Copy link

SparkQA commented Nov 17, 2015

Test build #46030 has finished for PR 9746 at commit 305e199.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):\n * class BitSet(numBits: Int) extends Serializable\n

@SparkQA
Copy link

SparkQA commented Nov 17, 2015

Test build #46063 has finished for PR 9746 at commit f69061d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Conflicts:
	core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala
asfgit pushed a commit that referenced this pull request Nov 18, 2015
This PR upgrade the version of RoaringBitmap to 0.5.10, to optimize the memory layout, will be much smaller when most of blocks are empty.

This PR is based on #9661 (fix conflicts), see all of the comments at #9661 .

Author: Kent Yao <yaooqinn@hotmail.com>
Author: Davies Liu <davies@databricks.com>
Author: Charles Allen <charles@allen-net.com>

Closes #9746 from davies/roaring_mapstatus.

(cherry picked from commit e33053e)
Signed-off-by: Davies Liu <davies.liu@gmail.com>
@asfgit asfgit closed this in e33053e Nov 18, 2015
@SparkQA
Copy link

SparkQA commented Nov 18, 2015

Test build #2083 has started for PR 9746 at commit f69061d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants