virajjasani commented on a change in pull request #3483:
URL: https://github.com/apache/hadoop/pull/3483#discussion_r716392145
##########
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/EncryptionZoneManager.java
##########
@@ -677,7 +677,7 @@ private boolean pathResolvesToId(final long zoneId, final
String zonePath)
* Cancels the currently-running re-encryption of the given encryption zone.
* If the given path is not the root of an encryption zone,
* an exception is thrown.
- * @param zoneIIP
+ * @param zoneIIP encryption zone inode path
Review comment:
same here
##########
File path:
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/EncryptionZoneManager.java
##########
@@ -645,8 +645,8 @@ private boolean pathResolvesToId(final long zoneId, final
String zonePath)
/**
* Re-encrypts the given encryption zone path. If the given path is not the
* root of an encryption zone, an exception is thrown.
- * @param zoneIIP
- * @param keyVersionName
+ * @param zoneIIP encryption zone inode path
Review comment:
nit: `encryption zone inodes in the path containing the file`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
