[ https://issues.apache.org/jira/browse/MESOS-9043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16529039#comment-16529039 ]
James Peach edited comment on MESOS-9043 at 7/1/18 10:07 AM: ------------------------------------------------------------- |[r/67794|https://reviews.apache.org/r/67794/]|Moved `validation::healthCheck` to common code.| |[r/67795|https://reviews.apache.org/r/67795/]|Moved `CheckInfo` validation to common code.| was (Author: jamespeach): |[r/67794|https://reviews.apache.org/r/67794/]|Moved `validation::healthCheck` to common code.| |[/r/67795|https://reviews.apache.org/r/67795/]|Moved `CheckInfo` validation to common code.| > Move check validators to the common validation library. > ------------------------------------------------------- > > Key: MESOS-9043 > URL: https://issues.apache.org/jira/browse/MESOS-9043 > Project: Mesos > Issue Type: Task > Components: build > Reporter: James Peach > Assignee: James Peach > Priority: Major > > The {{src/checks}} library contains some protobuf validation APIs that are > also used by the master. This creates a build dependency where the master > depends on the checks library but doesn't actually use the checks. We can > break this dependency by pushing the validators down into > {{src/common/validation.cpp}}. -- This message was sent by Atlassian JIRA (v7.6.3#76005)