Solr
  1. Solr
  2. SOLR-2579

UIMAUpdateRequestProcessor ignore error fails if text.length() < 0

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 3.2.1, 3.3, 4.0-ALPHA
    • Component/s: None
    • Labels:
      None

      Description

      If UIMAUpdateRequestProcessor is configured to ignore errors, an exception is raised when logging the error and text.length() < 100.

      if (solrUIMAConfiguration.isIgnoreErrors())
      log.warn(new StringBuilder("skip the text processing due to ")
      .append(e.getLocalizedMessage()).append(optionalFieldInfo)
      .append(" text=\"").append(text.substring(0, 100)).append("...\"").toString());
      else

      { throw new SolrException(ErrorCode.SERVER_ERROR, new StringBuilder("processing error: ") .append(e.getLocalizedMessage()).append(optionalFieldInfo) .append(" text=\"").append(text.substring(0, 100)).append("...\"").toString(), e); }

      I'm submitting a patch.

      1. SOLR-2579.patch
        3 kB
        Koji Sekiguchi
      2. SOLR-2579.patch
        2 kB
        Elmer Garduno

        Activity

        Hide
        Elmer Garduno added a comment -

        Fixes IndexOutOfBoundsException while logging a previous exception.

        Show
        Elmer Garduno added a comment - Fixes IndexOutOfBoundsException while logging a previous exception.
        Hide
        Koji Sekiguchi added a comment -

        Good catch, Elmer! I'm on traveling now. I'll look at this later today.

        Show
        Koji Sekiguchi added a comment - Good catch, Elmer! I'm on traveling now. I'll look at this later today.
        Hide
        Koji Sekiguchi added a comment -

        I added test. Will commit shortly.

        Show
        Koji Sekiguchi added a comment - I added test. Will commit shortly.
        Hide
        Koji Sekiguchi added a comment -

        trunk: Committed revision 1134163.
        3x: Committed revision 1134167.
        3.2: Committed revision 1134168.

        Thanks, Elmer!

        Show
        Koji Sekiguchi added a comment - trunk: Committed revision 1134163. 3x: Committed revision 1134167. 3.2: Committed revision 1134168. Thanks, Elmer!
        Hide
        Robert Muir added a comment -

        Bulk close for 3.3

        Show
        Robert Muir added a comment - Bulk close for 3.3

          People

          • Assignee:
            Koji Sekiguchi
            Reporter:
            Elmer Garduno
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development