[jira] [Commented] (HDDS-7835) Fix getKeyInfo API for snapshot paths.

["George Jahad (Jira)" <ji...@apache.org>]

    [ 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: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org