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

Noble Paul commented on SOLR-993:
---------------------------------

bq.Why shouldn't it work for the EvaluatorBag Functions? I am using the patch, 
as well as those functions...

Actually , I did not study the patch correctly. sorry about that.

I am not sure about one thing
{code:java}
  /*
   * Splits the name of a namespace into parts which contain only word 
characters.
   * The last part of a name may contain arbitrary characters, since it might 
refer to something other than a namespace.
   */
  private ArrayList<String> splitName(String name) {
{code}

can I add a name with '.' (dot)  in the end? . I guess it should be possible

bq.What if I have a namespace "a.b" containing a mapping with key "c.d", what 
does "a.b.c.d" refer to? The answer currently is it depends what other 
namespaces I have.

so , what should be the behavior? if I add a namespace "c.d" to "a.b" it should 
create a new namespace "c" inside "b" and put "d" into it? if we do  not do 
that it is still possible to have ambiguity.



> VariableResolverImpl addNamespace overwrites entire namespace instead of 
> adding
> -------------------------------------------------------------------------------
>
>                 Key: SOLR-993
>                 URL: https://issues.apache.org/jira/browse/SOLR-993
>             Project: Solr
>          Issue Type: Bug
>          Components: contrib - DataImportHandler
>    Affects Versions: 1.4
>            Reporter: Jared Flatow
>            Assignee: Shalin Shekhar Mangar
>             Fix For: 1.4
>
>         Attachments: SOLR-993.patch, SOLR-993b.patch
>
>   Original Estimate: 0.08h
>  Remaining Estimate: 0.08h
>
> The addNamespace method in VariableResolverImpl does not so much add the 
> namespace as overwrite it. 

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

Reply via email to