Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Apr 19, 2017

What changes were proposed in this pull request?

Checking a source parameter is asynchronous. When the query is created, it's not guaranteed that source has been created. This PR just increases the timeout of awaitTermination to ensure the parsing error is thrown.

How was this patch tested?

Jenkins

@SparkQA
Copy link

SparkQA commented Apr 19, 2017

Test build #75949 has finished for PR 17687 at commit a05a8c0.

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

@zsxwing
Copy link
Member Author

zsxwing commented Apr 19, 2017

Thanks! Merging to master and 2.2.

asfgit pushed a commit that referenced this pull request Apr 19, 2017
… by error

## What changes were proposed in this pull request?

Checking a source parameter is asynchronous. When the query is created, it's not guaranteed that source has been created. This PR just increases the timeout of awaitTermination to ensure the parsing error is thrown.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes #17687 from zsxwing/SPARK-20397.

(cherry picked from commit 4fea784)
Signed-off-by: Shixiong Zhu <[email protected]>
@asfgit asfgit closed this in 4fea784 Apr 19, 2017
@zsxwing zsxwing deleted the SPARK-20397 branch April 19, 2017 20:13
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
… by error

## What changes were proposed in this pull request?

Checking a source parameter is asynchronous. When the query is created, it's not guaranteed that source has been created. This PR just increases the timeout of awaitTermination to ensure the parsing error is thrown.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes apache#17687 from zsxwing/SPARK-20397.
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