[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13829908#comment-13829908
]
Noble Paul commented on SOLR-5436:
--
Why is it not yet resolved?
> Eliminate the 1500ms wa
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13829116#comment-13829116
]
ASF subversion and git services commented on SOLR-5436:
---
Commit 15442
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13829095#comment-13829095
]
Noble Paul commented on SOLR-5436:
--
Is there anything more we need to do before committing
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13829121#comment-13829121
]
ASF subversion and git services commented on SOLR-5436:
---
Commit 15442
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13828451#comment-13828451
]
Mark Miller commented on SOLR-5436:
---
bq. better than expecting the consumer of the API to
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13828443#comment-13828443
]
Noble Paul commented on SOLR-5436:
--
I think that exception handling is better in peek() me
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13828421#comment-13828421
]
Mark Miller commented on SOLR-5436:
---
You could of course explicitly add one - catch the c
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13828416#comment-13828416
]
Mark Miller commented on SOLR-5436:
---
bq. I don't see the exit condition from the loop in
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13828404#comment-13828404
]
Mark Miller commented on SOLR-5436:
---
bq. The Overseer went into a GC pause or something a
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13828400#comment-13828400
]
Noble Paul commented on SOLR-5436:
--
does this patch take care of this scenario?
The Overs
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13828021#comment-13828021
]
Mark Miller commented on SOLR-5436:
---
Ah, right - good point. We don't catch the watcher c
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13827997#comment-13827997
]
Noble Paul commented on SOLR-5436:
--
Yeah the Latch Watcher gets a callback , but it the pe
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13827981#comment-13827981
]
Mark Miller commented on SOLR-5436:
---
Keep in mind, that latch watcher will fire on zk con
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13827935#comment-13827935
]
Mark Miller commented on SOLR-5436:
---
bq. Because if the OverSeer changes and no state eve
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13827924#comment-13827924
]
Noble Paul commented on SOLR-5436:
--
I kept the 5sec it on purpose. Because if the OverSeer
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13827834#comment-13827834
]
Mark Miller commented on SOLR-5436:
---
I'm reviewing the latest patch.
> Eliminate the 150
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13826188#comment-13826188
]
Mark Miller commented on SOLR-5436:
---
I've looked it over, but I have to look closer at a
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13825482#comment-13825482
]
Mark Miller commented on SOLR-5436:
---
Yeah, I'll take a look today.
> Eliminate the 1500m
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13824778#comment-13824778
]
Noble Paul commented on SOLR-5436:
--
In my patch it would ensure that any event would get p
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13824735#comment-13824735
]
Mark Miller commented on SOLR-5436:
---
Shalin's original comment? Yeah, I suppose it does m
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13824732#comment-13824732
]
Noble Paul commented on SOLR-5436:
--
What if there are state change events happening in qui
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13824686#comment-13824686
]
Mark Miller commented on SOLR-5436:
---
bq. Originally we did not have a blocking lookup of
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13820228#comment-13820228
]
Noble Paul commented on SOLR-5436:
--
[~shalinmangar] Originally we did not have a blocking
[
https://issues.apache.org/jira/browse/SOLR-5436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13820082#comment-13820082
]
Shalin Shekhar Mangar commented on SOLR-5436:
-
As I understand it, the reason b
24 matches
Mail list logo