Github user dsmiley commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/423#discussion_r206539127
  
    --- Diff: 
solr/core/src/java/org/apache/solr/update/processor/TimeRoutedAliasUpdateProcessor.java
 ---
    @@ -272,7 +349,8 @@ private String 
findTargetCollectionGivenTimestamp(Instant docTimestamp) {
             return entry.getValue(); //found it
           }
         }
    -    return null; //not found
    +    throw new SolrException(SolrException.ErrorCode.BAD_REQUEST,
    +        "Doc " + id + " couldn't be routed with " + 
timeRoutedAlias.getRouteField() + "=" + docTimestamp);
    --- End diff --
    
    I think the error should say it's too old for the oldest collection


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to