[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-24 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16662965#comment-16662965 ] Josh Elser commented on RATIS-334: -- Had to tweak the protobuf-compiler pom stuff to account for the move

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-24 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16662909#comment-16662909 ] Josh Elser commented on RATIS-334: -- Thanks for the quick feedback, Sergey. Let's just plan to leave the

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-24 Thread Vladimir Rodionov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16662888#comment-16662888 ] Vladimir Rodionov commented on RATIS-334: - Someone (me or Sergey) should stop and wait until the

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-24 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16662883#comment-16662883 ] Sergey Soldatov commented on RATIS-334: --- bq. Just remove these from LogStateMachine instead of

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-24 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16662854#comment-16662854 ] Sergey Soldatov commented on RATIS-334: --- bq. equals(Object), toString(), and hashCode()? This class

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-24 Thread Vladimir Rodionov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16662762#comment-16662762 ] Vladimir Rodionov commented on RATIS-334: - np, I will rebase RATIS-369 on top of this. >

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-23 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16660867#comment-16660867 ] Josh Elser commented on RATIS-334: -- Either way to me. I can see arguments in either direction. >

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-22 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16660012#comment-16660012 ] Sergey Soldatov commented on RATIS-334: --- bq. Let's spin this out into its own Jira issue if you

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-22 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659968#comment-16659968 ] Josh Elser commented on RATIS-334: -- {quote}this class is ambiguous {quote} Let's spin this out into its

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-22 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659879#comment-16659879 ] Sergey Soldatov commented on RATIS-334: --- [~vrodionov] well, my point is that this class is ambiguous

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-22 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659600#comment-16659600 ] Josh Elser commented on RATIS-334: -- {quote}I don't quite understand what LogService class is supposed to

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-22 Thread Vladimir Rodionov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659342#comment-16659342 ] Vladimir Rodionov commented on RATIS-334: - LogService looks like a client, agree. I would leave

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-22 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659297#comment-16659297 ] Sergey Soldatov commented on RATIS-334: --- [~rajeshbabu] I don't quite understand what LogService

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-22 Thread Rajeshbabu Chintaguntla (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659241#comment-16659241 ] Rajeshbabu Chintaguntla commented on RATIS-334: --- [~sergey.soldatov] I have gone through the

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-19 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16657325#comment-16657325 ] Sergey Soldatov commented on RATIS-334: --- [~vrodionov], [~rajeshbabu] can you take a look.

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-17 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16653782#comment-16653782 ] Sergey Soldatov commented on RATIS-334: --- Ah, forgot to mention that Worker's state machine is

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-17 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16653766#comment-16653766 ] Sergey Soldatov commented on RATIS-334: --- This version includes: 1. proto files for both Log and Meta

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-09 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16644307#comment-16644307 ] Sergey Soldatov commented on RATIS-334: --- Just in case, to avoid any kind of confusion. The proto

[jira] [Commented] (RATIS-334) Implement server membership for LogService Metadata Service

2018-10-09 Thread Sergey Soldatov (JIRA)
[ https://issues.apache.org/jira/browse/RATIS-334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16644227#comment-16644227 ] Sergey Soldatov commented on RATIS-334: --- Attached wip patch for better understanding how