Skip to content

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