Skip to content

[SPARK-12492][SQL]Add missing SQLExecution.withNewExecutionId for hiveResultString#13689

Closed
zsxwing wants to merge 1 commit intoapache:masterfrom
zsxwing:pr13115
Closed

[SPARK-12492][SQL]Add missing SQLExecution.withNewExecutionId for hiveResultString#13689
zsxwing wants to merge 1 commit intoapache:masterfrom
zsxwing:pr13115

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Jun 15, 2016

What changes were proposed in this pull request?

Add missing SQLExecution.withNewExecutionId for hiveResultString so that queries running in spark-sql will be shown in Web UI.

Closes #13115

How was this patch tested?

Existing unit tests.

@SparkQA
Copy link

SparkQA commented Jun 15, 2016

Test build #60599 has finished for PR 13689 at commit b29b5ef.

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

@zsxwing
Copy link
Member Author

zsxwing commented Jun 15, 2016

/cc @andrewor14

@andrewor14
Copy link
Contributor

LGTM

@zsxwing
Copy link
Member Author

zsxwing commented Jun 15, 2016

Thanks, merging to master and 2.0

asfgit pushed a commit that referenced this pull request Jun 15, 2016
…veResultString

## What changes were proposed in this pull request?

Add missing SQLExecution.withNewExecutionId for hiveResultString so that queries running in `spark-sql` will be shown in Web UI.

Closes #13115

## How was this patch tested?

Existing unit tests.

Author: KaiXinXiaoLei <huleilei1@huawei.com>

Closes #13689 from zsxwing/pr13115.

(cherry picked from commit 3e6d567)
Signed-off-by: Shixiong Zhu <shixiong@databricks.com>
@asfgit asfgit closed this in 3e6d567 Jun 15, 2016
@zsxwing zsxwing deleted the pr13115 branch June 15, 2016 23:39
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