[ 
https://issues.apache.org/jira/browse/HDDS-13440?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wei-Chiu Chuang resolved HDDS-13440.
------------------------------------
    Resolution: Implemented

Resolving as implemented in the current repository. As noted in this issue, 
HDDS-12932 already fixed the directory-metadata SnapshotDiff NPE on master. 
DIRECTORY_TABLE_DEF now uses OmDirectoryInfo.getCodec(), and 
SnapshotDiffManager handles directory modifications separately from key objects.

Evidence at d0949f291791c399f3e54409fcb92656a80597a4:
* 
https://github.com/apache/ozone/blob/d0949f291791c399f3e54409fcb92656a80597a4/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/codec/OMDBDefinition.java#L273
* 
https://github.com/apache/ozone/blob/d0949f291791c399f3e54409fcb92656a80597a4/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/snapshot/SnapshotDiffManager.java#L1593

This resolves the master implementation tracking; it does not claim that a 
particular maintenance release received the backport.

> snapdiff operation could fail with a NullPointerException if a directory had 
> metadata changes
> ---------------------------------------------------------------------------------------------
>
>                 Key: HDDS-13440
>                 URL: https://issues.apache.org/jira/browse/HDDS-13440
>             Project: Apache Ozone
>          Issue Type: Sub-task
>          Components: Snapshot
>            Reporter: Sadanand Shenoy
>            Priority: Major
>
> [~sshenoy] discovered a bug where snapdiff operation could fail with a 
> NullPointerException (NPE) if a directory had metadata changes — such as ACL 
> updates or modification time updates — between snapshots.
>  
> The bug is unknowingly fixed in master branch by HDDS-12932.
>  
> Opening this Jira for posterity. Maybe we can consider including it in the 
> 2.0.1 release.



--
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