Skip to content

Conversation

@zjffdu
Copy link
Contributor

@zjffdu zjffdu commented Oct 17, 2017

What is this PR for?

Fixed the bug mentioned in #2554 (comment)

What type of PR is it?

[Bug Fix]

Todos

  • - Task

What is the Jira issue?

How should this be tested?

  • Unit test is added

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@zjffdu
Copy link
Contributor Author

zjffdu commented Oct 17, 2017

@Leemoonsoo Please help review. Thanks

@Leemoonsoo
Copy link
Member

LGTM!

@zjffdu
Copy link
Contributor Author

zjffdu commented Oct 18, 2017

Merge if no more discussion

@asfgit asfgit closed this in 24f8142 Oct 18, 2017
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