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-2846][SQL] Add configureInputJobPropertiesForStorageHandler to initialization of job conf #1927

Closed
wants to merge 1 commit into from

Conversation

alexliu68
Copy link

...al job conf

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@marmbrus
Copy link
Contributor

Thanks Alex! Please make the title [SPARK-2846][SQL] Add configureInputJobPropertiesForStorageHandler to initialization of job conf We have scripts that recognize that exact format and do auto linking with JIRA and stuff.

@marmbrus
Copy link
Contributor

ok to test

@SparkQA
Copy link

SparkQA commented Aug 13, 2014

QA tests have started for PR 1927. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18453/consoleFull

@SparkQA
Copy link

SparkQA commented Aug 13, 2014

QA results for PR 1927:
- This patch FAILED unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18453/consoleFull

@marmbrus
Copy link
Contributor

The test failure does not look related to this PR. We'll investigate.

@alexliu68 alexliu68 changed the title SPARK-SQL-2846 add configureInputJobPropertiesForStorageHandler to initi... [SPARK-2846][SQL] Add configureInputJobPropertiesForStorageHandler to initialization of job conf Aug 13, 2014
@marmbrus
Copy link
Contributor

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Aug 13, 2014

QA tests have started for PR 1927. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18485/consoleFull

@marmbrus
Copy link
Contributor

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Aug 14, 2014

QA tests have started for PR 1927. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18550/consoleFull

@SparkQA
Copy link

SparkQA commented Aug 14, 2014

QA results for PR 1927:
- This patch FAILED unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18550/consoleFull

@marmbrus
Copy link
Contributor

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Aug 20, 2014

QA tests have started for PR 1927 at commit e4bdc4c.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 20, 2014

QA tests have finished for PR 1927 at commit e4bdc4c.

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

asfgit pushed a commit that referenced this pull request Aug 20, 2014
… initialization of job conf

...al job conf

Author: Alex Liu <alex_liu68@yahoo.com>

Closes #1927 from alexliu68/SPARK-SQL-2846 and squashes the following commits:

e4bdc4c [Alex Liu] SPARK-SQL-2846 add configureInputJobPropertiesForStorageHandler to initial job conf

(cherry picked from commit d9e9414)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in d9e9414 Aug 20, 2014
@marmbrus
Copy link
Contributor

Thanks! I've merged this into master and 1.1.

xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
… initialization of job conf

...al job conf

Author: Alex Liu <alex_liu68@yahoo.com>

Closes apache#1927 from alexliu68/SPARK-SQL-2846 and squashes the following commits:

e4bdc4c [Alex Liu] SPARK-SQL-2846 add configureInputJobPropertiesForStorageHandler to initial job conf
szehon-ho pushed a commit to szehon-ho/spark that referenced this pull request Aug 7, 2024
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