[ https://issues.apache.org/jira/browse/SOLR-8120?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15055537#comment-15055537 ]
Cao Manh Dat edited comment on SOLR-8120 at 12/14/15 7:02 AM: -------------------------------------------------------------- Hi Varun, I dont think try-witch-resource is a good idea because we will have empty try block {code} try (IndexReader reader = solrCore.getIndexReaderFactory().newReader(tmpIndexDir, solrCore)) { //empty catch } catch (IOException e){ //do stuff here } {code} I also prefer an IOException (it tell specific what problem IndexFetcher encountered and handled perfectly by ReplicationHandler) to be thrown more than return false (which mean slave is already in sync). was (Author: caomanhdat): Hi Varun, I dont think try-witch-resource is a good idea because we will have empty try block ```java try (IndexReader reader = solrCore.getIndexReaderFactory().newReader(tmpIndexDir, solrCore)) { //empty catch } catch (IOException e){ //do stuff here } ``` I also prefer an IOException (it tell specific what problem IndexFetcher encountered and handled perfectly by ReplicationHandler) to be thrown more than return false (which mean slave is already in sync). > Validate index files on replica before old index is deleted > ----------------------------------------------------------- > > Key: SOLR-8120 > URL: https://issues.apache.org/jira/browse/SOLR-8120 > Project: Solr > Issue Type: Improvement > Components: replication (java) > Reporter: Shalin Shekhar Mangar > Labels: difficulty-easy, impact-high > Fix For: 5.4, Trunk > > Attachments: SOLR-8120.patch > > > During replication, if a full sync is necessary then the detection of > corruption will happen after the index has been cleaned up (refer to > solrCore.getDirectoryFactory().remove(indexDir); before calling > openNewSearcherAndUpdateCommitPoint()) in IndexFetcher. > SOLR-8119 will make this issue obsolete but until then we should perform > trivial verification (maybe just open a reader and close?) before the replica > blows away its old index? -- 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