Skip to content

Conversation

@nishkamravi2
Copy link
Contributor

nishkamravi2 and others added 30 commits June 3, 2014 15:28
…extFiles

The prefix "file:" is missing in the string inserted as key in HashMap
…multiplier (redone to resolve merge conflicts)
…nravi

Conflicts:
	yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala
…nravi

Conflicts:
	yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientArguments.scala
	yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala
	yarn/common/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
nishkamravi2 and others added 20 commits March 24, 2015 16:42
…nravi

Conflicts:
	launcher/src/main/java/org/apache/spark/launcher/AbstractCommandBuilder.java
…nravi

Conflicts:
	bin/spark-class
	bin/spark-class2.cmd
	launcher/src/main/java/org/apache/spark/launcher/AbstractCommandBuilder.java
…nravi

Conflicts:
	yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala
…k.kryoserializer.buffer use int conversion since fractions are not permissible
…nravi

Conflicts:
	core/src/main/scala/org/apache/spark/SparkConf.scala
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@srowen
Copy link
Member

srowen commented Jun 1, 2015

That's fine. This does not warrant a JIRA.
(PS why do your PRs have a huge merge history? you'd normally just branch from an updated master right?)

@nishkamravi2 nishkamravi2 changed the title [SPARK-7959][DOC] Minor modification to Streaming docs with regards to parallel data receiving [DOC] Minor modification to Streaming docs with regards to parallel data receiving Jun 1, 2015
@nishkamravi2
Copy link
Contributor Author

Yeah, the JIRA in the title seems unnecessary (removed). With regards to the merge history: not sure, will have to look into it.

@tdas
Copy link
Contributor

tdas commented Jun 1, 2015

LGTM other than the commit history.
Can you just just checkout the spark github master branch and cherry-pick your change (commit 46e8c03) on it. That should keep the commit history clean.

@srowen
Copy link
Member

srowen commented Jun 1, 2015

FWIW I keep master in my fork up to date with git pull upstream master and git push origin master. Then whenever you make a branch, just branch from your master. Have you been merging into your master? It should just track the upstream one directly with no variance.

asfgit pushed a commit that referenced this pull request Jun 1, 2015
…ata receiving

pwendell tdas

Author: Nishkam Ravi <[email protected]>
Author: nishkamravi2 <[email protected]>
Author: nravi <[email protected]>

Closes #6544 from nishkamravi2/master_nravi and squashes the following commits:

46e8c03 [Nishkam Ravi] Slight modification to streaming docs

(cherry picked from commit e7c7e51)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in e7c7e51 Jun 1, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
…ata receiving

pwendell tdas

Author: Nishkam Ravi <[email protected]>
Author: nishkamravi2 <[email protected]>
Author: nravi <[email protected]>

Closes apache#6544 from nishkamravi2/master_nravi and squashes the following commits:

46e8c03 [Nishkam Ravi] Slight modification to streaming docs
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
…ata receiving

pwendell tdas

Author: Nishkam Ravi <[email protected]>
Author: nishkamravi2 <[email protected]>
Author: nravi <[email protected]>

Closes apache#6544 from nishkamravi2/master_nravi and squashes the following commits:

46e8c03 [Nishkam Ravi] Slight modification to streaming docs
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