[ 
https://issues.apache.org/jira/browse/SOLR-6920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14309788#comment-14309788
 ] 

Mark Miller commented on SOLR-6920:
-----------------------------------

bq. 3

I'm not following - what are you seeing exactly? I read it as 'if a file was 
not checksummed, download anyway if it matches this file list'.

bq. || (!compareResult.checkSummed && (((String) 
file.get(NAME)).endsWith(".si") || ((String) file.get(NAME)).endsWith(".liv")  
|| ((String) file.get(NAME)).startsWith("segments_")))

If the file is checksummed, we can rely on the checksum. What am I missing that 
you are seeing?

> During replication use checksums to verify if files are the same
> ----------------------------------------------------------------
>
>                 Key: SOLR-6920
>                 URL: https://issues.apache.org/jira/browse/SOLR-6920
>             Project: Solr
>          Issue Type: Bug
>          Components: replication (java)
>            Reporter: Varun Thacker
>            Assignee: Mark Miller
>            Priority: Critical
>         Attachments: SOLR-6920.patch, SOLR-6920.patch, SOLR-6920.patch, 
> SOLR-6920.patch, SOLR-6920.patch, SOLR-6920.patch
>
>
> Currently we check if an index file on the master and slave is the same by 
> checking if it's name and file length match. 
> With LUCENE-2446 we now have a checksums for each index file in the segment. 
> We should leverage this to verify if two files are the same.
> Places like SnapPuller.isIndexStale and SnapPuller.downloadIndexFiles should 
> check against the checksum also.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to