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

Posted to reviews@yunikorn.apache.org by GitBox <gi...@apache.org> on 2020/10/05 23:23:09 UTC

[GitHub] [incubator-yunikorn-core] yangwwei commented on pull request #207: [YUNIKORN-405] Added checksum validation

yangwwei commented on pull request #207:
URL: https://github.com/apache/incubator-yunikorn-core/pull/207#issuecomment-703940708


   > If we will not add the checksum validation, we might be in the case that the user is performing some unwanted changes, like deleting some changes made by an another user, without even noticing it
   
   Correct. But I think such logic should be built outside of YuniKorn. The client-side stores the configuration and use that as the source of truth, the client-side can avoid concurrent updates, stale configs, etc, which seems to be an easier way for integration.


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