Skip to content

Conversation

@syedhashmi
Copy link

This change adds a new partitioner which allows users
to specify # of keys per partition.

This change adds a new partitioner which allows users
to specify # of keys per partition.
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@syedhashmi syedhashmi changed the title [SPARK-1784] Add a new partitioner [SPARK-1784] Add a new partitioner to allow specifying # of keys per partition May 12, 2014
@asfgit asfgit closed this in 92cebad May 14, 2014
asfgit pushed a commit that referenced this pull request May 14, 2014
…partition

This change adds a new partitioner which allows users
to specify # of keys per partition.

Author: Syed Hashmi <[email protected]>

Closes #721 from syedhashmi/master and squashes the following commits:

4ca94cc [Syed Hashmi] [SPARK-1784] Add a new partitioner

(cherry picked from commit 92cebad)
Signed-off-by: Reynold Xin <[email protected]>
@srowen
Copy link
Member

srowen commented May 14, 2014

@rxin Oh interestingly we were just having a side conversation about this.

Is it going to be an issue that the partitioner does not consistently assign the same key to the same partition? I didn't think this would work. The assignments entirely local to the instance of the partitioner. Does this not also maybe yield different result in a multi-threaded context, even with the same input?

@pwendell
Copy link
Contributor

This was merged by mistake by @rxin (I just spoke with him). I am going to revert this. Unfortunately it also broke the master build.

@pwendell
Copy link
Contributor

@rxin merged this in lieu of #729 - so I'm going to merge that one now.

@rxin
Copy link
Contributor

rxin commented May 14, 2014

Thanks for doing this.

On Tuesday, May 13, 2014, Patrick Wendell [email protected] wrote:

@rxin https://github.com/rxin merged this in lieu of #729https://github.com/apache/spark/pull/729\- so I'm going to merge that one now.


Reply to this email directly or view it on GitHubhttps://github.com//pull/721#issuecomment-43046527
.

pdeyhim pushed a commit to pdeyhim/spark-1 that referenced this pull request Jun 25, 2014
…partition

This change adds a new partitioner which allows users
to specify # of keys per partition.

Author: Syed Hashmi <[email protected]>

Closes apache#721 from syedhashmi/master and squashes the following commits:

4ca94cc [Syed Hashmi] [SPARK-1784] Add a new partitioner
agirish pushed a commit to HPEEzmeral/apache-spark that referenced this pull request May 5, 2022
turboFei pushed a commit to turboFei/spark that referenced this pull request Nov 6, 2025
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