Right, that's the way I was leaning. I could argue that there's some value in stopping the recursion, one would continue on past the core.properties.bak file into the conf and data dirs, but I really don't think that's persuasive.
'...file to something besides "core.properties" too.' should have read '...file to something besides "core.properties" instead.' So no code change it is. Erick On Mon, Jul 1, 2013 at 9:47 AM, Mark Miller <[email protected]> wrote: > > On Jul 1, 2013, at 9:37 AM, Erick Erickson <[email protected]> > wrote: > > > What do people think about allowing an option in "core.properties" files > like ignoreMe=true|false (default to false)? I was thinking it might be > useful to temporarily have Solr ignore a core without having to physically > remove it. > > It's a good idea. > > > > > I suppose you could really do the same thing by renaming the file to > something besides "core.properties" too. > > But I think that negates the need for any additional code. Adding a .bak > seem like a fine way to handle this and it adds no code. > > So I'd vote for no change, other than perhaps a doc note that mentions > this use case or something. > > - Mark > > > > > Worth the bother? > > > > Erick > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
