[ 
https://issues.apache.org/jira/browse/HDDS-7835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17681156#comment-17681156
 ] 

George Jahad commented on HDDS-7835:
------------------------------------

 

I'm pretty sure I fixed this last week: 
https://issues.apache.org/jira/browse/HDDS-7765

 

But since this ticket was created after that fix, I'm not positive.  
[~sadanand_shenoy]  Have you noticed this problem since my fix was merged?

> Fix getKeyInfo API for snapshot paths.
> --------------------------------------
>
>                 Key: HDDS-7835
>                 URL: https://issues.apache.org/jira/browse/HDDS-7835
>             Project: Apache Ozone
>          Issue Type: Sub-task
>            Reporter: Sadanand Shenoy
>            Priority: Major
>
> HDDS-7230 introduced a new getKeyInfo API in OM code, however it doesn't take 
> care of snapshot paths as it doesn't use OMMetadataReader abstraction and 
> returns null as it searches for the snapshot path in the active DB . This in 
> turn fails the getFileStatus code as it calls the getKeyInfoAPI
> *RpcClient.java* ~~ line 1913
> {code:java}
> final OmKeyInfo keyInfo;
> if (omVersion.compareTo(OzoneManagerVersion.OPTIMIZED_GET_KEY_INFO) >= 0) {
>   keyInfo = ozoneManagerClient.getKeyInfo(keyArgs, false)
>       .getKeyInfo();
>   if (!keyInfo.isFile()) {
>     throw new OMException(keyName + " is not a file.",
>         OMException.ResultCodes.NOT_A_FILE);
>   }
> } else {
>   keyInfo = ozoneManagerClient.lookupFile(keyArgs);
> } {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to