szetszwo commented on PR #5532: URL: https://github.com/apache/hadoop/pull/5532#issuecomment-1499923239
> ... when call reaches here, the snapshot id is CURRENT_STATE_ID, which represent current state. Why are we proceeding to clean subtree originally with CURRENT_STATE_ID. Could you provide more details on that? ... The might be a bug. `prior` and `snapshot` are supposed to be the prior and the current Snapshot IDs. Because of rename, the subtree was moved from somewhere to there. The logic in `getPriorSnapshot` and `getSelfSnapshot` may be incorrect. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org