steveloughran commented on issue #1576: Hadoop 16520 dynamodb ms version race refactor. URL: https://github.com/apache/hadoop/pull/1576#issuecomment-537611585 I should add -overall patch LGTM; these are very minor changes. Impressed that the tag update means that you get this version resilience automatically, one thought there: should we think about tagging the table explicitly with some "s3guard" string, e.g "s3guard-version" - makes it clearer to admins looking at tables which are guarded
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org