Skip to content

Conversation

@cku328
Copy link
Contributor

@cku328 cku328 commented Nov 7, 2019

What changes were proposed in this pull request?

Fixed typo in param description, replacing
* @param acl ozone acl top be added.
with
* @param acl ozone acl to be added.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-2369

How was this patch tested?

Ran UTs

Copy link
Member

@elek elek left a comment

Choose a reason for hiding this comment

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

+1 Thanks the the fix @cku328

Will merge it soon.

((I am very surprised that we had multiple instances from the same typo))

@elek elek closed this in 384fee0 Nov 7, 2019
@elek
Copy link
Member

elek commented Nov 7, 2019

@cku328 I would like to assign the jira (to have the credit) but can't see your jira username. Can you please add any comment to https://issues.apache.org/jira/browse/HDDS-2369 with your jira user?

Thanks.

@cku328
Copy link
Contributor Author

cku328 commented Nov 7, 2019

Thanks @elek for the review, I am adding the user name in Jira.

@elek
Copy link
Member

elek commented Nov 8, 2019

Thanks, I added your user to the HDDS Jira project as contributor. Now you can assign any issue to yourself and you can be an assignee...

smengcl added a commit to smengcl/hadoop-ozone that referenced this pull request Aug 19, 2025
Removes native library includes from rocks-native/pom.xml and adds an antrun plugin in dist/pom.xml to copy native libraries to the Ozone distribution. Updates the shell script to include the new library path in JAVA_LIBRARY_PATH for relevant commands. This improves the packaging and loading of native libraries in Ozone.

Rocks tools native lib should not be inside jar apache#123 (-HDDS-11591. Copy dependencies when building each module)
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