honeymoose/OpenSearch
1
0
Fork 0
mirror of https://github.com/honeymoose/OpenSearch.git synced 2025-02-09 06:25:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixing the error message to report the number of docs correctly for each node (#22515)

Browse Source 
There is a bug in the error message that is thrown if the number of docs differs between the source and target shards when recovering a shard with a syncId. The source and target doc counts are swapped around.

Closes #21893
This commit is contained in:
henakamaMSFT 2017-01-10 00:57:57 -08:00 committed by Simon Willnauer
parent dc81afd4f8
commit 72ec3d2661
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
core/src/main/java/org/elasticsearch/indices/recovery/RecoverySourceHandler.java
View File

@ -222,7 +222,7 @@ public class RecoverySourceHandler {
final long numDocsSource = recoverySourceMetadata.getNumDocs(); final long numDocsSource = recoverySourceMetadata.getNumDocs();
if (numDocsTarget != numDocsSource) { if (numDocsTarget != numDocsSource) {
throw new IllegalStateException("try to recover " + request.shardId() + " from primary shard with sync id but number " + throw new IllegalStateException("try to recover " + request.shardId() + " from primary shard with sync id but number " +
"of docs differ: " + numDocsTarget + " (" + request.sourceNode().getName() + ", primary) vs " + numDocsSource "of docs differ: " + numDocsSource + " (" + request.sourceNode().getName() + ", primary) vs " + numDocsTarget
+ "(" + request.targetNode().getName() + ")"); + "(" + request.targetNode().getName() + ")");
} }
// we shortcut recovery here because we have nothing to copy. but we must still start the engine on the target. // we shortcut recovery here because we have nothing to copy. but we must still start the engine on the target.