You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-dev@lucene.apache.org by "Noble Paul (JIRA)" <ji...@apache.org> on 2009/12/04 05:25:20 UTC

[jira] Created: (SOLR-1621) Deprecate deployments w/o solr.xml

Deprecate deployments w/o solr.xml
----------------------------------

                 Key: SOLR-1621
                 URL: https://issues.apache.org/jira/browse/SOLR-1621
             Project: Solr
          Issue Type: New Feature
    Affects Versions: 1.5
            Reporter: Noble Paul
             Fix For: 1.5


supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789439#action_12789439 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

You've deprecated handleAliasAction, but you have also made it so its not called with the default handleRequestBody.

I think its safer to just remove handleAliasAction - someone that overrode handleAliasAction but kept the default handleRequestBody would be in a for a silent nasty surprise anyway. Better to get a compile time error.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786449#action_12786449 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

bq. what are the supporting reasons?

It gives transparency. And the user will be conscious about the existence of solr.xml . like tomcat offers the default web.xml which users usually don't edit. But tells the users that it exists and certain behaviour is driven from that .
Users usually take the example and make the necessary changes. So technically they don't 'add' it.  Anyway this is a minor technicality. 

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787303#action_12787303 ] 

Noble Paul edited comment on SOLR-1621 at 12/8/09 4:29 AM:
-----------------------------------------------------------

bq.if they cause too much grief, we always have the option to remove.

Do we really have a usecase for ALIAS ? if there is no no compelling enough usecase we should consider removing it. there is a lot of code which is there just becaus eof this alias feature

      was (Author: noble.paul):
    bq.if they cause too much grief, we always have the option to remove.

Do we really have a usecase for ALIAS ? if there is no not compelling enough usecase we should consider removing it. there is a lot of code which is there just becaus eof this alias feature
  
> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Miller updated SOLR-1621:
------------------------------

    Attachment:     (was: SOLR-1621.patch)

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787908#action_12787908 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

bq.I'm good with yanking ALIAS for now.

I am happy to remove a lot of complexity from CoreContainer which was introduced by ALIAS. When we implemented SOLR-1293 (internally) we disabled ALIAS so that the implementation is simple. 
Let us revisit ALIAS later and make it simpler. 


> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Miller updated SOLR-1621:
------------------------------

    Attachment: SOLR-1621.patch

Whoops - wrong solr.xml example in wrong page - had core1 for the name.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786481#action_12786481 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

{code}
It should show something like

Admin DEFAULT_CORE (alias :"" )
Admin core1 (alias : "c2","c3")
{code}

+1 - that would be nice.

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Miller updated SOLR-1621:
------------------------------

    Attachment: SOLR-1621.patch

Looks like it has the same issue I ran into -

The solrConfigFilename doesn't work right now (used by tests and web.xml override).

What I did is add another load method that also takes the config filename (or not if its null). I don't like adding the extra methods for this, so I'm not sure this is the best solution, but its a solution to get started with.

That will get the NoCacheHeaderTest to pass.

I also dropped a solr.xml into example to get started there.


> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789443#action_12789443 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

Also, its confusing the way the Alias test is commented out - shouldn't we just remove to be less confusing? It will still be in subversion if we decide we want to bring it back.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Yonik Seeley (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787038#action_12787038 ] 

Yonik Seeley commented on SOLR-1621:
------------------------------------

Aliases were marked as experimental - if they cause too much grief, we always have the option to remove.


> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787968#action_12787968 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

also deprecated specifying solrconfig from web.xml. We may not need it anymore because we should not support 2 ways of doing the same

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786461#action_12786461 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

Just to clear this up:

This issue is not a proposal to deprecate the current single core mode - its a proposal to allow specifying the current single core mode from solr.xml.

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786783#action_12786783 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

hi Mark could you update the patch to trunk?

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786343#action_12786343 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

bq.continue supporting no solr.xml without deprecation. If a user is not going to customize anything,

It is better to be consistent in the long run. There is no harm if the user drops in this default solr.xml whether he customizes it or not. So , I believe we should get rid of it 2 releases later.

bq.One of the minor issues is that the default page that shows all of the cores doesn't care about aliases -

another thing I wish to change in this is we should store the aliases also in the CoreDescriptor . That makes it simpler to show details. 

It should show something like 

Admin DEFAULT_CORE (alias :"" )
Admin core1 (alias : "c2","c3")



> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789302#action_12789302 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

I'd like to take a quick look - will report back.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790128#action_12790128 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

bq. me too. But this looked like an exception. This is rarely used (if at all) and according to me is a bad feature because it needs editing web.xml .

Thats fine - as long as we have good warnings, I'm fine with removing it - but we *must* have a replacement for the tests that count on this feature.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>            Assignee: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787303#action_12787303 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

bq.if they cause too much grief, we always have the option to remove.

Do we really have a usecase for ALIAS ? if there is no not compelling enough usecase we should consider removing it. there is a lot of code which is there just becaus eof this alias feature

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Noble Paul updated SOLR-1621:
-----------------------------

    Attachment: SOLR-1621.patch

The necessary code changes. 

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Kay Kay (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12785800#action_12785800 ] 

Kay Kay commented on SOLR-1621:
-------------------------------

+1 . Thanks - This is long due, the moment we get into the multi-core world. 

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Eric Pugh (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12785932#action_12785932 ] 

Eric Pugh commented on SOLR-1621:
---------------------------------

Glad to see this bug.  In the Solr book we mentioned that this was a very likely thing, and that even if you think you only need 1 core, starting out with a multicore setup is the way to go!  Especially when you toss in all the great management features of multiple cores!

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Noble Paul updated SOLR-1621:
-----------------------------

    Attachment: SOLR-1621.patch

it works now even after the alias command . I still think we should remove the 'alias' command. It is a fancy feature which adds too much of complexity into ref counting of cores

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786407#action_12786407 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

bq. It is better to be consistent in the long run.

Why - what are the supporting reasons? Making current users put in a boiler plate file in the future doesn't appear to have any benefits to me. If they want to use settings in the file, they can add the file, but just to add it for the sake of adding it is not very appealing.

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790079#action_12790079 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

The patch looks fine . I plan to commit it soon

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>            Assignee: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786643#action_12786643 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

Nice! Essentially the same as my little patch, but I hadn't done the alias stuff. Reviewing now.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790145#action_12790145 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

bq. The patch looks fine . I plan to commit it soon

I still think we need to do something about handleAliasAction - possibly just remove it. It's not really a deprecation like it says - if someone overrode it, its no longer called. Better to get a hard error.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>            Assignee: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Noble Paul updated SOLR-1621:
-----------------------------

    Attachment: SOLR-1621.patch

the index pages are fixed

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Hoss Man (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787704#action_12787704 ] 

Hoss Man commented on SOLR-1621:
--------------------------------

The nice thing about the *idea* of ALIAS is that you can have cores with very explicit names (ie: catalog_v1, catalog_v2, etc...) and then you can have aliases like "LIVE" and "EXPERIMENTAL" that you can move around as needed.  Similar things can be accomplished with the SWAP command, but it's more limiting.

That said: the last time i tired using ALIAS it was such a confusing pain in the ass because of the way the original name was still tracked separately from the list of aliases, even if that name had been taken over by another core) i couldn't bring myself to use it -- using SWAP and keeping track of the logical names externally was less confusing.

If we can make ALIAS work well, it will kick ass -- but we can always yank it for now if it's in our way, and add it back again later if someone comes up with a good way to do it.  (it was probably a mistake to try and treat names an aliases as equals in the first place ... looking up a core by a "string" is probably the only use cases where they should be treated equally, all of the other CoreAdmin commands should really differentiate.)

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789467#action_12789467 ] 

Mark Miller edited comment on SOLR-1621 at 12/11/09 7:13 PM:
-------------------------------------------------------------

This patch reinstates a fix for the SolrDispatcher config name override so that all tests pass again.
It also changes DEFAULT_CORE to use a final static constant rather than be repeated.


Personally, I also almost think that removing the alias feature should be its own issue.

(*EDIT* okay, I see you made that a sub-issue - nice)

      was (Author: markrmiller@gmail.com):
    This patch reinstates a fix for the SolrDispatcher config name override so that all tests pass again.
It also changes DEFAULT_CORE to use a final static constant rather than be repeated.


Personally, I also almost think that removing the alias feature should be its own issue.
  
> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790185#action_12790185 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

I shall soon give a final patch w/ the removal of ALIAS.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>            Assignee: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Noble Paul resolved SOLR-1621.
------------------------------

    Resolution: Fixed

committed r890675

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>            Assignee: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12785791#action_12785791 ] 

Noble Paul edited comment on SOLR-1621 at 12/4/09 7:09 AM:
-----------------------------------------------------------

The proposed design.

Going forward for the single core example we will add a solr.xml to the $SOLR_HOME/solr directory whose content will look like

{code:xml}
<?xml version="1.0" encoding="UTF-8" ?>
<solr persistent="false">
  <cores adminPath="/admin/cores">
    <!- "DEFAULT_CORE" is a special name . Solr will automatically create an alias "" for this name-->
    <core name="DEFAULT_CORE" instanceDir="." />    
  </cores>
</solr>
{code}

the core will be accessible by both the names

* http://host:port/solr/ ("" is an alias for DEFAULT_CORE )
* http://host:port/solr/DEFAULT_CORE

All coreadmin actions can be performed using the core name as DEFAULT_CORE. So all coreadmin features will work for single core as well

For legacy users who have single core deployments no change needs to be done. if the solr.xml is missing we will use a default solr.xml whose content is same as above. So everything will be backward compatible. We can mark the deployments w/o solr.xml as deprecated and the we can remove it completely later.





      was (Author: noble.paul):
    The proposed design.

Going forward for the single core example we will add a solr.xml to the $SOLR_HOME/solr directory whose content will look like

{code:xml}
<?xml version="1.0" encoding="UTF-8" ?>
<solr persistent="false">
  <cores adminPath="/admin/cores">
    <core name="DEFAULT_CORE" instanceDir="." />    
  </cores>
</solr>
{code}

the core will be accessible by both the names

* http://host:port/solr/ ("" is an alias for DEFAULT_CORE )
* http://host:port/solr/DEFAULT_CORE

All coreadmin actions can be performed using the core name as DEFAULT_CORE. So all coreadmin features will work for single core as well

For legacy users who have single core deployments no change needs to be done. if the solr.xml is missing we will use a default solr.xml whose conmtent is same as above. So everything will be backward compatible. We can mark the deployments w/o solr.xml as deprecated and the we can remove it completely later.




  
> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Yonik Seeley (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787712#action_12787712 ] 

Yonik Seeley commented on SOLR-1621:
------------------------------------

I'm good with yanking ALIAS for now.

But I think it would be nice to have a single hard-coded alias (perhaps just at the level of dispatch filter) that can treat an existing core as the default core (w/o having to name it something specific like DEFAULT_CORE.  As long as we can have a normal core with a normal name like "music", with the ability to use it with legacy URLs.


> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12785791#action_12785791 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

The proposed design.

Going forward for the single core example we will add a solr.xml to the $SOLR_HOME/solr directory whose content will look like

{code:xml}
<?xml version="1.0" encoding="UTF-8" ?>
<solr persistent="false">
  <cores adminPath="/admin/cores">
    <core name="DEFAULT_CORE" instanceDir="." />    
  </cores>
</solr>
{code}

the core will be accessible by both the names

* http://host:port/solr/ ("" is an alias for DEFAULT_CORE )
* http://host:port/solr/DEFAULT_CORE

All coreadmin actions can be performed using the core name as DEFAULT_CORE. So all coreadmin features will work for single core as well

For legacy users who have single core deployments no change needs to be done. if the solr.xml is missing we will use a default solr.xml whose conmtent is same as above. So everything will be backward compatible. We can mark the deployments w/o solr.xml as deprecated and the we can remove it completely later.





> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786454#action_12786454 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

bq. And the user will be conscious about the existence of solr.xml .

They will be aware because we will include it with example.

bq. So technically they don't 'add' it.

New users won't add it, but old users will have to it - generally with no benefit. If you are just running single core, being conscious of solr.xml is not very useful. Forcing older users to add the file, just to make them conscious off it, is not very persuasive in my opinion. Its more config for the user with no benefit.

bq. Anyway this is a minor technicality

Its something that we will have to come to consensus on.

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Noble Paul updated SOLR-1621:
-----------------------------

    Attachment: SOLR-1621.patch

Throw a message saying 'ALIAS is removed' when ALIAS is invoked

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>            Assignee: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Miller updated SOLR-1621:
------------------------------

    Attachment: SOLR-1621.patch

Its up to trunk - looks like its an issue with $id - I'm so sick of running into that. Here is a patch with the $id created by the patch replaced with the actual $id string - that should make it apply.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789689#action_12789689 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

I guess deprecating ALIAS can be committed first so that the changes for this issue will be smaller and more manageable

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12785791#action_12785791 ] 

Noble Paul edited comment on SOLR-1621 at 12/4/09 7:09 AM:
-----------------------------------------------------------

The proposed design.

Going forward for the single core example we will add a solr.xml to the $SOLR_HOME/solr directory whose content will look like

{code:xml}
<?xml version="1.0" encoding="UTF-8" ?>
<solr persistent="false">
  <cores adminPath="/admin/cores">
    <!-- "DEFAULT_CORE" is a special name . Solr will automatically create an alias "" for this name-->
    <core name="DEFAULT_CORE" instanceDir="." />    
  </cores>
</solr>
{code}

the core will be accessible by both the names

* http://host:port/solr/ ("" is an alias for DEFAULT_CORE )
* http://host:port/solr/DEFAULT_CORE

All coreadmin actions can be performed using the core name as DEFAULT_CORE. So all coreadmin features will work for single core as well

For legacy users who have single core deployments no change needs to be done. if the solr.xml is missing we will use a default solr.xml whose content is same as above. So everything will be backward compatible. We can mark the deployments w/o solr.xml as deprecated and the we can remove it completely later.





      was (Author: noble.paul):
    The proposed design.

Going forward for the single core example we will add a solr.xml to the $SOLR_HOME/solr directory whose content will look like

{code:xml}
<?xml version="1.0" encoding="UTF-8" ?>
<solr persistent="false">
  <cores adminPath="/admin/cores">
    <!- "DEFAULT_CORE" is a special name . Solr will automatically create an alias "" for this name-->
    <core name="DEFAULT_CORE" instanceDir="." />    
  </cores>
</solr>
{code}

the core will be accessible by both the names

* http://host:port/solr/ ("" is an alias for DEFAULT_CORE )
* http://host:port/solr/DEFAULT_CORE

All coreadmin actions can be performed using the core name as DEFAULT_CORE. So all coreadmin features will work for single core as well

For legacy users who have single core deployments no change needs to be done. if the solr.xml is missing we will use a default solr.xml whose content is same as above. So everything will be backward compatible. We can mark the deployments w/o solr.xml as deprecated and the we can remove it completely later.




  
> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Miller updated SOLR-1621:
------------------------------

    Attachment: SOLR-1621.patch

You mark setSolrConfigFilename as deprecated, but it simply doesn't work anymore either - thats not really deprecation. I feel that deprecating solrconfig-filename should be its own issue so thats a more clear entry in CHANGES. And it should prob be deprecated rather than turned off out of the blue. I don't suppose its a big deal either way though.

You have also changed CoreContainer#getCores - but that change is unrelated to this patch. I really feal an issue should just stick to the issue - it makes commits more clear. That change (which isn't back compat anyway), should be its own commit.

This latest update removes a bunch of the unrelated changes.

The patch is still unfinished though - you removed my fix to use the solrconfig override without replacing it with an alternative fix. As a result, some tests don't use the right config now, and in particular, NoCacheHeaderTest is back to failing.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786051#action_12786051 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

I wrote a little patch trying this out -

One of the minor issues is that the default page that shows all of the cores doesn't care about aliases - I think thats something we should address -

add a getNonAliasCores (with a different name) or something. Otherwise you see the following on the index.jsp page:

Welcome to Solr!

Admin DEFAULT_CORE
Admin DEFAULT_CORE

Based on the feedback in the discussion that spawned this, we should also consider not deprecating solr.xml. I think we should
put one in the default example, but continue supporting no solr.xml without deprecation. If a user is not going to customize anything,
we don't gain much by forcing them to add the default solr.xml that we can assume anyway - and we still get all of the advantages of this issue.

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Noble Paul updated SOLR-1621:
-----------------------------

    Attachment: SOLR-1621.patch

with ALIAS deprecated

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789988#action_12789988 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

bq.Personally, I'd prefer a form of deprecation first.

me too.  But this looked like an exception. This is rarely used (if at all) and according to me is a bad feature because it needs editing web.xml . 

bq.At a minimum, it needs to be its own issue so that the warning to users that they need to change what they are doing is very clear.

There is another issue now SOLR-1647 .I should change the language. they are really not deprecations. 

Ok . I can go both ways on this. we can keep the feature deprecated and remove it later or remove it right away. My preference would be to remove it.



> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786677#action_12786677 ] 

Mark Miller edited comment on SOLR-1621 at 12/6/09 7:39 PM:
------------------------------------------------------------

Whoops - wrong solr.xml example in last patch - had core1 for the name.

      was (Author: markrmiller@gmail.com):
    Whoops - wrong solr.xml example in wrong page - had core1 for the name.
  
> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Miller updated SOLR-1621:
------------------------------

    Attachment: SOLR-1621.patch

This patch reinstates a fix for the SolrDispatcher config name override so that all tests pass again.
It also changes DEFAULT_CORE to use a final static constant rather than be repeated.


Personally, I also almost think that removing the alias feature should be its own issue.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Noble Paul reassigned SOLR-1621:
--------------------------------

    Assignee: Noble Paul

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>            Assignee: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789675#action_12789675 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

bq.You mark setSolrConfigFilename as deprecated, but it simply doesn't work anymore either - thats not really deprecation. I feel that deprecating solrconfig-filename should be its own issue so thats a more clear entry in CHANGES

I shall open another issue for this.  The setSolrConfigFilename() did not work for multicore . So it does not make sense for us to bend over backwards to make it work in this and add too many overloaded methods. Anyway, users won't need 



> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786625#action_12786625 ] 

Noble Paul edited comment on SOLR-1621 at 12/6/09 3:26 PM:
-----------------------------------------------------------

The necessary code changes for corecontainer. 

      was (Author: noble.paul):
    The necessary code changes. 
  
> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Noble Paul (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789116#action_12789116 ] 

Noble Paul commented on SOLR-1621:
----------------------------------

hi . If everyone is OK I would just cleanup the patch and commit this shortly..

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Miller updated SOLR-1621:
------------------------------

    Summary: Allow current single core deployments to be specified by solr.xml  (was: Deprecate deployments w/o solr.xml)

I think this description is more in line with what the proposal is - while deprecation may occur, its not the primary work to be done - this is an improvement and issue whether we decided to deprecate or not - we are also not deprecating "single core deployments" - we would be deprecating a lack of solr.xml.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786459#action_12786459 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

If you don't see solr.xml, you just assume the very simple default solr.xml - the same as solr will.

bq. you would have to change the directory structure somewhere

Thats the point of this issue - you won't have to change the directory structure.

bq. introduce a new configuration file, and only then you would be able to add more databases

You will only introduce a new config file if you decided to forgo it to start - thats fair. The example will come with it.

The purpose of this issue is to make the current single core specifiable from solr.xml - that doesn't mean you need an explicit solr.xml - any tool and solr can assume the most basic solr.xml that would be the default anyway :  private final static String SINGLE_CORE_CONFIG = "<solr persistent=\"false\"><cores adminPath=\"/admin/cores\"><core name=\"DEFAULT_CORE\" instanceDir=\".\" /></cores></solr>";

I'll let it go - others that don't wan't it required (from the zookeeper issue) can chime in, or I'll just let it go.

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789926#action_12789926 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

bq. The setSolrConfigFilename() did not work for multicore . 

But it did work for single core, and we are emulating that. I agree that it should be removed, but perhaps not in how. I think it should go away with warning - and I don't think we should mark something as deprecated when it just no longer works - deprecation means the method should still work. Also, I'm not sure if you have realized it yet, but tests currently rely on this functionality - it cannot be pulled without replacing the functionality for tests in some manner.

bq. So it does not make sense for us to bend over backwards to make it work in this and add too many overloaded methods. 

It doesn't require bending over backwards. It doesn't even require overloaded methods (see my latest patch) - that was just one method.


bq. Anyway, users won't need

Its not about users needing this - they won't need it because they can now use solr.xml - its about functionality that someone is currently counting on simply disappearing. Personally, I'd prefer a form of deprecation first. At a minimum, it needs to be its own issue so that the warning to users that they need to change what they are doing is *very* clear.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Uri Boness (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786456#action_12786456 ] 

Uri Boness commented on SOLR-1621:
----------------------------------

In addition to the awareness I believe it's also consistent. Having two deferent configuration mechanisms for single and multi-core eventually adds to the complexity. A consistent directory layout and one single access point to the configuration (that is, solr.xml) simplifies things - user don't need to keep in their mind "oh... this is multi-core... so it uses this layout" or "oh... it's only one core, so there is not solr.xml just this directory". Instead they get a consistent view of the directory/configuration structure  whether the use multi-core or not. 

Also note it also helps in developing tools for Solr as you don't need to deal with different configurations types and scenarios.

Thinks of RDBMS's... MySQL for example. Imagine MySQL would come with a default configuration of a single database. But when you wanted to add more databases, you would have to change the directory structure somewhere, introduce a new configuration file, and only then you would be able to add more databases. In that respect, Solr can be seen RDBMS and Solr Core can be seen as a Database within this RDBMS. 

> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12789447#action_12789447 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

Lastly : there are a couple little unrelated changes in this patch.

1. The change to the import statement in CoreDescriptor, but no other changes to the file.
2. The change in index.jsp from self closing tags to closing link tags - this has no apparent purpose, so I think it shouldn't be part of this commit.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Miller updated SOLR-1621:
------------------------------

    Attachment: SOLR-1621.patch

This solves the closing core too many times issue you can have by just simply checking if a core is closed before closing it in shutdown.

By the way, why not allow removing the DEFAULT_CORE? Shouldn't it be treated like any other core, other than the special property of getting a "" alias?

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SOLR-1621) Deprecate deployments w/o solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786459#action_12786459 ] 

Mark Miller edited comment on SOLR-1621 at 12/5/09 6:19 PM:
------------------------------------------------------------

If you don't see solr.xml, you just assume the very simple default solr.xml - the same as solr will.

bq. you would have to change the directory structure somewhere

Thats the point of this issue - you won't have to change the directory structure.

bq. introduce a new configuration file, and only then you would be able to add more databases

You will only introduce a new config file if you decided to forgo it to start - thats fair. The example will come with it.

bq. Having two deferent configuration mechanisms for single and multi-core eventually adds to the complexity

There won't be two configuration mechanisms - if you want to do core configuration you will need the file - in either case, for a release or two, solr will assume a solr.xml that works with no solr.xml in the dir - taking that away is more pain for those that don't have or want it than helpful in my opinion. There won't be two configuration mechanisms in either case.

The purpose of this issue is to make the current single core specifiable from solr.xml - that doesn't mean you need an explicit solr.xml - any tool and solr can assume the most basic solr.xml that would be the default anyway :  private final static String SINGLE_CORE_CONFIG = "<solr persistent=\"false\"><cores adminPath=\"/admin/cores\"><core name=\"DEFAULT_CORE\" instanceDir=\".\" /></cores></solr>";

I'll let it go - others that don't wan't it required (from the zookeeper issue) can chime in, or I'll just let it go.

      was (Author: markrmiller@gmail.com):
    If you don't see solr.xml, you just assume the very simple default solr.xml - the same as solr will.

bq. you would have to change the directory structure somewhere

Thats the point of this issue - you won't have to change the directory structure.

bq. introduce a new configuration file, and only then you would be able to add more databases

You will only introduce a new config file if you decided to forgo it to start - thats fair. The example will come with it.

The purpose of this issue is to make the current single core specifiable from solr.xml - that doesn't mean you need an explicit solr.xml - any tool and solr can assume the most basic solr.xml that would be the default anyway :  private final static String SINGLE_CORE_CONFIG = "<solr persistent=\"false\"><cores adminPath=\"/admin/cores\"><core name=\"DEFAULT_CORE\" instanceDir=\".\" /></cores></solr>";

I'll let it go - others that don't wan't it required (from the zookeeper issue) can chime in, or I'll just let it go.
  
> Deprecate deployments w/o solr.xml
> ----------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12787031#action_12787031 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

bq. the index pages are fixed

how are they fixed?

I think one problem with how you are handling alias' is that it only works with alias' defined in solr.xml?

If you use the request handler to create an alias (with the alias command), I still don't think that works properly.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786666#action_12786666 ] 

Mark Miller commented on SOLR-1621:
-----------------------------------

We should try and hit SOLR-1626 with this too since we are doing alias work anyway.

We either need to drop too many closes as a logging error in SolrCore, or set things up so that we only close primary core names and not aliases.

> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SOLR-1621) Allow current single core deployments to be specified by solr.xml

Posted by "Mark Miller (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/SOLR-1621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12786675#action_12786675 ] 

Mark Miller edited comment on SOLR-1621 at 12/6/09 7:28 PM:
------------------------------------------------------------

This solves the closing core too many times issue you can have by just simply checking if a core is closed before closing it in shutdown.

By the way, why not allow removing the DEFAULT_CORE? Shouldn't it be treated like any other core, other than the special property of getting a "" alias?

* edit *

We still need to solve listing alias' as primary cores in index.jsp as well.

      was (Author: markrmiller@gmail.com):
    This solves the closing core too many times issue you can have by just simply checking if a core is closed before closing it in shutdown.

By the way, why not allow removing the DEFAULT_CORE? Shouldn't it be treated like any other core, other than the special property of getting a "" alias?
  
> Allow current single core deployments to be specified by solr.xml
> -----------------------------------------------------------------
>
>                 Key: SOLR-1621
>                 URL: https://issues.apache.org/jira/browse/SOLR-1621
>             Project: Solr
>          Issue Type: New Feature
>    Affects Versions: 1.5
>            Reporter: Noble Paul
>             Fix For: 1.5
>
>         Attachments: SOLR-1621.patch, SOLR-1621.patch, SOLR-1621.patch
>
>
> supporting two different modes of deployments is turning out to be hard. This leads to duplication of code. Moreover there is a lot of confusion on where do we put common configuration. See the mail thread http://markmail.org/message/3m3rqvp2ckausjnf

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.