umamaheswararao commented on a change in pull request #3127:
URL: https://github.com/apache/ozone/pull/3127#discussion_r816094083
##########
File path:
hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/io/BlockOutputStreamEntry.java
##########
@@ -223,7 +223,7 @@ long getLength() {
* Gets the block token that is used to authenticate during the write.
* @return the block token for writing the data
*/
- Token<OzoneBlockTokenIdentifier> getToken() {
Review comment:
Not sure it's a good idea to make this public as we return token object?
I have an idea to avoid make them public. Create a small helper class in the
same package in test and add getToken and getCurrentPosition APIs and return
the actual.
class name at test: *.client.io.BlockStreamAccessor
--
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]
