Skip to content
This repository was archived by the owner on Jan 9, 2020. It is now read-only.

Conversation

@hex108
Copy link

@hex108 hex108 commented Jan 4, 2018

What changes were proposed in this pull request?

Fix #594

How was this patch tested?

Unit test and manual test.

@liyinan926
Copy link
Member

@hex108 apache#20148 has been merged. Can you refactor this PR? Thanks!

@hex108
Copy link
Author

hex108 commented Jan 5, 2018

@liyinan926 Ah, might it be better to cherry-pick your patch at apache#20148?

@liyinan926
Copy link
Member

Yep, that sounds good to me.

@liyinan926
Copy link
Member

I'm gonna cherry-pick the changes in apache#20148 and create a new PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants