Skip to content

Conversation

@kevinw66
Copy link
Contributor

@kevinw66 kevinw66 commented Aug 5, 2022

Description of PR

How was this patch tested?

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
  • Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

Copy link
Contributor

@sekikn sekikn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for addressing this @kevinw66, I left some comments.

@kevinw66
Copy link
Contributor Author

kevinw66 commented Aug 5, 2022

Thanks for addressing this @kevinw66, I left some comments.

Thanks @sekikn , I fixed it

@sekikn
Copy link
Contributor

sekikn commented Aug 5, 2022

+1, tested on CentOS 7 x86_64.

@sekikn sekikn merged commit 60578d4 into apache:master Aug 5, 2022
@sekikn
Copy link
Contributor

sekikn commented Aug 5, 2022

Merged into master. Thanks @kevinw66!

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