Skip to content

[Storm 1715] Jedis Default Host#1345

Closed
darionyaphet wants to merge 3 commits intoapache:masterfrom
darionyaphet:1715
Closed

[Storm 1715] Jedis Default Host#1345
darionyaphet wants to merge 3 commits intoapache:masterfrom
darionyaphet:1715

Conversation

@darionyaphet
Copy link
Copy Markdown
Contributor

Useing Jedis Protocol.DEFAULT_HOST to replace DEFAULT_HOST :)

STORM-1715

@HeartSaVioR
Copy link
Copy Markdown
Contributor

@darionyaphet
+1
Could you rebase and squash your commits into one?

@darionyaphet
Copy link
Copy Markdown
Contributor Author

@HeartSaVioR It's seems I make a mistake. pull request #1 is a merge request I commit long time ago. I couldn't checkout it . Maybe I should remove this repo and recommit this PR ?

@HeartSaVioR
Copy link
Copy Markdown
Contributor

@darionyaphet Yeah, if git rebase master doesn't work, you may want to create a new branch and try to cherry pick b3c0e2d.

@darionyaphet
Copy link
Copy Markdown
Contributor Author

@HeartSaVioR thanks for you help . I will close this pull request and reopen another . 👍

bipinprasad pushed a commit to bipinprasad/storm that referenced this pull request Oct 17, 2019
YSTORM-6263: Removes numa hard coding testing line
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.

2 participants