[jira] [Comment Edited] (MESOS-4610) MasterContender/MasterDetector should be loadable as modules
[ https://issues.apache.org/jira/browse/MESOS-4610?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15197665#comment-15197665 ] ANURAG SINGH edited comment on MESOS-4610 at 3/31/16 5:16 PM: -- We've proceeded further with the review and here's the updated list of changes: https://reviews.apache.org/r/44287/ : Added MasterContender and MasterDetector abstract classes. https://reviews.apache.org/r/44288/ : Changed MasterDetector/Contender namespace. https://reviews.apache.org/r/44543/ : Removed unnecessary MasterContender and MasterDetector definitions. https://reviews.apache.org/r/44544/ : Moved contender and detector definitions into separate directories. https://reviews.apache.org/r/44545/ : Instead of keeping standalone and zookeper contender/detector class definitions and implementations in the same file, separated them. Also made the necessary changes in users of class headers to point to the new locations. https://reviews.apache.org/r/44546/ : Moved functions in promises to a common header file. https://reviews.apache.org/r/44547/ : Added functions in promises to the collect header. https://reviews.apache.org/r/44289/ : Added support for contender and detector modules. https://reviews.apache.org/r/44669/ : Added createFromModule methods to MasterContender and MasterDetector. https://reviews.apache.org/r/44670/ : Added master_detector and master_contender flags. was (Author: anurag.prakash.singh): We've proceeded further with the review and here's the updated list of changes: https://reviews.apache.org/r/44287/: Added MasterContender and MasterDetector abstract classes. https://reviews.apache.org/r/44288/: Changed MasterDetector/Contender namespace. https://reviews.apache.org/r/44543/: Removed unnecessary MasterContender and MasterDetector definitions. https://reviews.apache.org/r/44544/: Moved contender and detector definitions into separate directories. https://reviews.apache.org/r/44545/: Instead of keeping standalone and zookeper contender/detector class definitions and implementations in the same file, separated them. Also made the necessary changes in users of class headers to point to the new locations. https://reviews.apache.org/r/44546/: Moved functions in promises to a common header file. https://reviews.apache.org/r/44547/: Added functions in promises to the collect header. https://reviews.apache.org/r/44289/: Added support for contender and detector modules. https://reviews.apache.org/r/44669/: Added createFromModule methods to MasterContender and MasterDetector. https://reviews.apache.org/r/44670/: Added master_detector and master_contender flags. > MasterContender/MasterDetector should be loadable as modules > > > Key: MESOS-4610 > URL: https://issues.apache.org/jira/browse/MESOS-4610 > Project: Mesos > Issue Type: Improvement > Components: master >Reporter: Mark Cavage >Assignee: Mark Cavage > Labels: mesosphere > > Currently mesos depends on Zookeeper for leader election and notification to > slaves, although there is a C++ hierarchy in the code to support alternatives > (e.g., unit tests use an in-memory implementation). From an operational > perspective, many organizations/users do not want to take a dependency on > Zookeeper, and use an alternative solution to implementing leader election. > Our organization in particular, very much wants this, and as a reference > there have been several requests from the community (see referenced tickets) > to replace with etcd/consul/etc. > This ticket will serve as the work effort to modularize the > MasterContender/MasterDetector APIs such that integrators can build a > pluggable solution of their choice; this ticket will not fold in any > implementations such as etcd et al., but simply move this hierarchy to be > fully pluggable. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Comment Edited] (MESOS-4610) MasterContender/MasterDetector should be loadable as modules
[ https://issues.apache.org/jira/browse/MESOS-4610?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15197665#comment-15197665 ] ANURAG SINGH edited comment on MESOS-4610 at 3/28/16 3:22 PM: -- We've proceeded further with the review and here's the updated list of changes: https://reviews.apache.org/r/44287/: Added MasterContender and MasterDetector abstract classes. https://reviews.apache.org/r/44288/: Changed MasterDetector/Contender namespace. https://reviews.apache.org/r/44543/: Removed unnecessary MasterContender and MasterDetector definitions. https://reviews.apache.org/r/44544/: Moved contender and detector definitions into separate directories. https://reviews.apache.org/r/44545/: Instead of keeping standalone and zookeper contender/detector class definitions and implementations in the same file, separated them. Also made the necessary changes in users of class headers to point to the new locations. https://reviews.apache.org/r/44546/: Moved functions in promises to a common header file. https://reviews.apache.org/r/44547/: Added functions in promises to the collect header. https://reviews.apache.org/r/44289/: Added support for contender and detector modules. https://reviews.apache.org/r/44669/: Added createFromModule methods to MasterContender and MasterDetector. https://reviews.apache.org/r/44670/: Added master_detector and master_contender flags. was (Author: anurag.prakash.singh): We've proceeded further with the review and here's the updated list of changes: https://reviews.apache.org/r/44287/ https://reviews.apache.org/r/44288/ https://reviews.apache.org/r/44543/ https://reviews.apache.org/r/44544/ https://reviews.apache.org/r/44545/ https://reviews.apache.org/r/44546/ https://reviews.apache.org/r/44547/ https://reviews.apache.org/r/44289/ https://reviews.apache.org/r/44669/ https://reviews.apache.org/r/44670/ > MasterContender/MasterDetector should be loadable as modules > > > Key: MESOS-4610 > URL: https://issues.apache.org/jira/browse/MESOS-4610 > Project: Mesos > Issue Type: Improvement > Components: master >Reporter: Mark Cavage >Assignee: Mark Cavage > Labels: mesosphere > > Currently mesos depends on Zookeeper for leader election and notification to > slaves, although there is a C++ hierarchy in the code to support alternatives > (e.g., unit tests use an in-memory implementation). From an operational > perspective, many organizations/users do not want to take a dependency on > Zookeeper, and use an alternative solution to implementing leader election. > Our organization in particular, very much wants this, and as a reference > there have been several requests from the community (see referenced tickets) > to replace with etcd/consul/etc. > This ticket will serve as the work effort to modularize the > MasterContender/MasterDetector APIs such that integrators can build a > pluggable solution of their choice; this ticket will not fold in any > implementations such as etcd et al., but simply move this hierarchy to be > fully pluggable. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Comment Edited] (MESOS-4610) MasterContender/MasterDetector should be loadable as modules
[ https://issues.apache.org/jira/browse/MESOS-4610?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15136149#comment-15136149 ] Klaus Ma edited comment on MESOS-4610 at 2/7/16 5:22 AM: - [~mcavage], this proposal makes sense to me :). Would you help to summary the implementation detail before RR? The code diff is really huge. BTW, I think you can drop an email to dev@ to find a shepherd for this JIRA. was (Author: klaus1982): [~mcavage], this proposal makes sense to me :). Would you help to summary the implementation detail before RR? The code diff is really huge. > MasterContender/MasterDetector should be loadable as modules > > > Key: MESOS-4610 > URL: https://issues.apache.org/jira/browse/MESOS-4610 > Project: Mesos > Issue Type: Improvement > Components: master >Reporter: Mark Cavage > > Currently mesos depends on Zookeeper for leader election and notification to > slaves, although there is a C++ hierarchy in the code to support alternatives > (e.g., unit tests use an in-memory implementation). From an operational > perspective, many organizations/users do not want to take a dependency on > Zookeeper, and use an alternative solution to implementing leader election. > Our organization in particular, very much wants this, and as a reference > there have been several requests from the community (see referenced tickets) > to replace with etcd/consul/etc. > This ticket will serve as the work effort to modularize the > MasterContender/MasterDetector APIs such that integrators can build a > pluggable solution of their choice; this ticket will not fold in any > implementations such as etcd et al., but simply move this hierarchy to be > fully pluggable. -- This message was sent by Atlassian JIRA (v6.3.4#6332)