[
https://issues.apache.org/jira/browse/SOLR-13189?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16758720#comment-16758720
]
Hoss Man commented on SOLR-13189:
---------------------------------
bq. markmiller: Here is a hack to that test.
yeah, fair enough -- sorry, I wasn't trying to be dismissive of your help, ...
i guess i was just hoping for a less complicated (from the perspective of test
writers) solution that we could show case as the gold standard of how to
(generically) "wait for recovery" after (potentially) injecting failures ...
but i'm not in a rush to re-add TestInjection back into
TestStressCloudBlindAtomicUpdates -- it's a "nice to have" but not something I
care about enough to get over my general feeling of ickiness at needing call
{{Thread.sleep}} in a loop that much : )
> Need reliable example (Test) of how to use TestInjection.failReplicaRequests
> ----------------------------------------------------------------------------
>
> Key: SOLR-13189
> URL: https://issues.apache.org/jira/browse/SOLR-13189
> Project: Solr
> Issue Type: Sub-task
> Security Level: Public(Default Security Level. Issues are Public)
> Reporter: Hoss Man
> Priority: Major
> Attachments: SOLR-13189.patch, SOLR-13189.patch, SOLR-13189.patch
>
>
> We need a test that reliably demonstrates the usage of
> {{TestInjection.failReplicaRequests}} and shows what steps a test needs to
> take after issuing updates to reliably "pass" (finding all index updates that
> succeeded from the clients perspective) even in the event of an (injected)
> replica failure.
> As things stand now, it does not seem that any test using
> {{TestInjection.failReplicaRequests}} passes reliably -- *and it's not clear
> if this is due to poorly designed tests, or an indication of a bug in
> distributed updates / LIR*
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
