Skip to content

Conversation

@insidedctm
Copy link
Contributor

Comments preceding toMessage method state: "The edge partition is encoded in the lower

  • 30 bytes of the Int, and the position is encoded in the upper 2 bytes of the Int.". References to bytes should be changed to bits.

This contribution is my original work and I license the work to the Spark project under it's open source license.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@srowen
Copy link
Member

srowen commented Sep 14, 2015

LGTM

@andrewor14
Copy link
Contributor

Merging into master 1.5.

@andrewor14
Copy link
Contributor

Actually, for some reason I couldn't merge this patch. @srowen do you want to try?

@srowen
Copy link
Member

srowen commented Sep 14, 2015

Hm failed for me too. I'll investigate if I get a sec tonight, def. tomorrow if not

@rxin
Copy link
Contributor

rxin commented Sep 15, 2015

I just merged this. Thanks!

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.

5 participants