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-2452] Create a new valid for each instead of using lineId. #1441

Closed

Conversation

ScrapCodes
Copy link
Member

No description provided.

@SparkQA
Copy link

SparkQA commented Jul 16, 2014

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

@SparkQA
Copy link

SparkQA commented Jul 16, 2014

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

@SparkQA
Copy link

SparkQA commented Jul 16, 2014

QA results for PR 1441:
- This patch PASSES 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/16726/consoleFull

@SparkQA
Copy link

SparkQA commented Jul 16, 2014

QA results for PR 1441:
- This patch PASSES 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/16727/consoleFull

@ScrapCodes ScrapCodes changed the title SPARK-2452, create a new valid for each instead of using lineId. [SPARK-2452] Create a new valid for each instead of using lineId. Jul 22, 2014
@pwendell
Copy link
Contributor

Okay merging this into master. For the 1.0 branch we just reverted the fix.

@asfgit asfgit closed this in 81fec99 Jul 22, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Author: Prashant Sharma <prashant@apache.org>

Closes apache#1441 from ScrapCodes/SPARK-2452/multi-statement and squashes the following commits:

26c5c72 [Prashant Sharma] Added a test case.
7e8d28d [Prashant Sharma] SPARK-2452, create a new valid for each  instead of using lineId, because Line ids can be same sometimes.
@ScrapCodes ScrapCodes deleted the SPARK-2452/multi-statement branch June 3, 2015 06:02
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.

3 participants