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-3705][SQL]add case for VoidObjectInspector to cover NullType #2552

Closed
wants to merge 1 commit into from

Conversation

scwf
Copy link
Contributor

@scwf scwf commented Sep 27, 2014

add case for VoidObjectInspector in inspectorToDataType

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@scwf scwf changed the title [SPARK-3705]add case for VoidObjectInspector to cover NullType [SPARK-3705][SQL]add case for VoidObjectInspector to cover NullType Sep 27, 2014
@marmbrus
Copy link
Contributor

ok to test

@SparkQA
Copy link

SparkQA commented Sep 27, 2014

QA tests have started for PR 2552 at commit 453d892.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 27, 2014

Tests timed out after a configured wait of 120m.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20922/

@scwf
Copy link
Contributor Author

scwf commented Sep 27, 2014

@marmbrus , it seems all PR of SQL tests timed out

@scwf
Copy link
Contributor Author

scwf commented Sep 28, 2014

I tested for the time out issue, #1689 lead to this issue, but have not found the root cause

@scwf
Copy link
Contributor Author

scwf commented Sep 30, 2014

@marmbrus, test this and #2551, they failed due to time out, thanks

@scwf
Copy link
Contributor Author

scwf commented Oct 1, 2014

@pwendell, can you help to test this and #2551, i did not have the access for jenkins test

@marmbrus
Copy link
Contributor

marmbrus commented Oct 1, 2014

Sorry, I've been really busy and haven't had time to review PRs. Starting tests now. Pending those this LGTM.

@SparkQA
Copy link

SparkQA commented Oct 1, 2014

QA tests have started for PR 2552 at commit 453d892.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 1, 2014

QA tests have finished for PR 2552 at commit 453d892.

  • This patch passes unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class PStatsParam(AccumulatorParam):

@marmbrus
Copy link
Contributor

marmbrus commented Oct 1, 2014

Thanks! Merged to master.

@asfgit asfgit closed this in f315fb7 Oct 1, 2014
@scwf scwf deleted the inspectorToDataType branch October 3, 2014 13:17
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