You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2019/11/05 15:08:49 UTC

[GitHub] [hadoop-ozone] smengcl commented on issue #108: HDDS-1987. Fix listStatus API

smengcl commented on issue #108: HDDS-1987. Fix listStatus API
URL: https://github.com/apache/hadoop-ozone/pull/108#issuecomment-549864119
 
 
   Thanks @anuengineer for the comment.
   
   As I discussed with @bharatviswa504 last week, since DB iterator is grabbed after TableCache iteration, there shouldn't really be a missing key from the result.
   
   But there might be other cases of inconsistency. A large chunk of logic in this patch is learnt from HDDS-1986. So what could happen here could also happen in `listKeys`.
   
   I agree we should refactor `KeyManagerImpl` later.
   
   I've rebased my patch to latest master(trunk) hence the force-push.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org