Skip to content

Conversation

@caofangkun
Copy link
Contributor

No description provided.

@revans2
Copy link
Contributor

revans2 commented Jan 12, 2016

I thought I merged this in, but I guess I messed something up with the merger. +1

@ptgoetz
Copy link
Member

ptgoetz commented Jan 12, 2016

@revans2 looks like this was originally #996, which had the requisite +1s, but needed an upmerge. I think @caofangkun just opened a new pull request and closed #996.

+1

@haohui
Copy link

haohui commented Jan 12, 2016

+1

@asfgit asfgit merged commit 64a57a3 into apache:master Jan 13, 2016
@caofangkun caofangkun deleted the storm-1453-1 branch January 15, 2016 01:43
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.

5 participants