SqlEntityProcessor should do something (like throw an error) if DOT_PATTERN is 
not matched
------------------------------------------------------------------------------------------

                 Key: SOLR-1818
                 URL: https://issues.apache.org/jira/browse/SOLR-1818
             Project: Solr
          Issue Type: Bug
          Components: contrib - DataImportHandler
            Reporter: Mark Miller
            Priority: Trivial
             Fix For: 1.5


Looking like we should do something if DOT_PATTERN does not match a primary key 
(rather then get the resulting nullpointer exception) - I'm not really up on 
DIH, so I'm not sure if that is something we never expect to see, but even in 
that case it might be nice to add an else throw illegalstate or something with 
a "we should never get here" comment - just for future DIH devs.

{code}
      Object val = context.resolve("dataimporter.delta." + primaryKey);
      if (val == null) {
        Matcher m = DOT_PATTERN.matcher(primaryKey);
        if (m.find()) {
          val = context.resolve("dataimporter.delta." + m.group(1));
        }
      }
      sb.append(primaryKey).append(" = ");
      if (val instanceof Number) {
        sb.append(val.toString());
      } else {
        sb.append("'").append(val.toString()).append("'");
      }
{code}

-- 
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