Skip to content

Conversation

@SaintBacchus
Copy link
Contributor

No description provided.

@SparkQA
Copy link

SparkQA commented Jan 6, 2016

Test build #48846 has finished for PR 10617 at commit 70a12b6.

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

@zsxwing
Copy link
Member

zsxwing commented Jan 6, 2016

Thanks @SaintBacchus LGTM. Merging to master, 1.6, 1.5

@asfgit asfgit closed this in 19e4e9f Jan 6, 2016
asfgit pushed a commit that referenced this pull request Jan 6, 2016
…lt root path to gain the streaming batch url.

Author: huangzhaowei <[email protected]>

Closes #10617 from SaintBacchus/SPARK-12672.
asfgit pushed a commit that referenced this pull request Jan 6, 2016
…lt root path to gain the streaming batch url.

Author: huangzhaowei <[email protected]>

Closes #10617 from SaintBacchus/SPARK-12672.
@zsxwing
Copy link
Member

zsxwing commented Jan 6, 2016

As this patch doesn't add basePath to the url, I think #10618 is better. So I'm going to revert this one.

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