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

Shawn Heisey commented on SOLR-4910:
------------------------------------

{quote}
corea is renamed coreb. Is coreb the persisted name?
corea and coreb are swapped, what is persisted?
{quote}

I know a little bit about this, as I use core swapping in conjunction with full 
index rebuilds.  Here's my opinion.  Things that should be persisted any time 
there's a change:

* Properties whose value has changed.  In the case of a rename or swap, that 
would be the core name(s).
* Properties that are explicitly stated in solr.xml or the core.properties that 
is being overwritten.
* Properties whose default value will be different in a future version of Solr.

Except as described in the list above, I don't think that unspecified defaults 
should be written.  When I took my solr.xml from 3.5.0 and put it on 
4.1-SNAPSHOT, I suddenly had new data in my solr.xml after a core swap, the 
settings for LotsOfCores.  I don't think that should have been written, because 
it wasn't there in the first place.

                
> solr.xml persistence is completely broken
> -----------------------------------------
>
>                 Key: SOLR-4910
>                 URL: https://issues.apache.org/jira/browse/SOLR-4910
>             Project: Solr
>          Issue Type: Bug
>    Affects Versions: 5.0, 4.4
>            Reporter: Erick Erickson
>            Assignee: Erick Erickson
>            Priority: Blocker
>
> I'm working on SOLR-4862 (persisting a created core doesn't preserve some 
> values) and at least compared to 4.3 code, persisting to solr.xml is 
> completely broken.
> I learned to hate persistence while working on SOLR-4196 & etc. and I'm glad 
> it's going away. I frequently got lost in implicit properties (they're easy 
> to persist and shouldn't be), what should/shouldn't be persisted (e.g. the 
> translated ${var:default} or the original), and it was a monster, so don't 
> think I'm nostalgic for the historical behavior.
> Before I dive back in I want to get some idea whether or not the current 
> behavior was intentional or not, I don't want to go back into that junk only 
> to undo someone else's work.
> Creating a new core (collection2 in my example) with persistence turned on in 
> solr.xml for instance changes the original definition for collection1 (stock 
> 4.x as of tonight) from this:
> <core name="collection1" instanceDir="collection1" shard="${shard:}" 
> collection="${collection:collection1}" config="${solrconfig:solrconfig.xml}" 
> schema="${schema:schema.xml}"
>           coreNodeName="${coreNodeName:}"/>
> to this:
>   <core loadOnStartup="true" shard="${shard:}" instanceDir="collection1/" 
> transient="false" name="collection1" dataDir="data/" 
> collection="${collection:collection1}">
>       <property name="name" value="collection1"/>
>       <property name="config" value="solrconfig.xml"/>
>       <property name="solr.core.instanceDir" value="solr/collection1/"/>
>       <property name="transient" value="false"/>
>       <property name="schema" value="schema.xml"/>
>       <property name="loadOnStartup" value="true"/>
>       <property name="solr.core.schemaName" value="schema.xml"/>
>       <property name="solr.core.name" value="collection1"/>
>       <property name="solr.core.dataDir" value="data/"/>
>       <property name="instanceDir" value="collection1/"/>
>       <property name="solr.core.configName" value="solrconfig.xml"/>
>     </core>
> So, there are two questions:
> 1> what is correct for 4.x?
> 2> do we care at all about 5.x?
> As much as I hate to say it, I think that we need to go back to the 4.3 
> behavior. It might be as simple as not persisting in the <property> tags 
> anything already in the original definition. Not quite sure what to put where 
> in the newly-created core though, I suspect that the compact <core + attribs> 
> would be best (assuming there's no <property> tag already in the definition. 
> I really hate the mix of attributes on the <core> tag and <property> tags, 
> wish we had one or the other....

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to