Solr
  1. Solr
  2. SOLR-3403

Move logging of deprecated Analysis Factory inside Factory itself

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0-ALPHA
    • Component/s: Schema and Analysis
    • Labels:
      None

      Description

      Currently runtime logging of deprecated Factories is handled by BaseTokenStreamFactory which looks odd since the logging comes from this class instead of the Factories themselves. We should just move the logging down to the deprecated class.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        5d 19h 42m 1 Chris Male 30/Apr/12 00:33
        Resolved Resolved Closed Closed
        375d 10h 7m 1 Uwe Schindler 10/May/13 10:40
        Uwe Schindler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Chris Male made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Chris Male [ cmale ]
        Fix Version/s 4.0 [ 12314992 ]
        Resolution Fixed [ 1 ]
        Hide
        Chris Male added a comment -

        Resolved in trunk.

        Show
        Chris Male added a comment - Resolved in trunk.
        Chris Male made changes -
        Field Original Value New Value
        Attachment SOLR-3403.patch [ 12524828 ]
        Hide
        Chris Male added a comment -

        Patch which moves all logging in the factories down to the particular classes and removes all logging support in the abstract classes.

        Any Factory that wants to log now must declare its own Logger.

        Show
        Chris Male added a comment - Patch which moves all logging in the factories down to the particular classes and removes all logging support in the abstract classes. Any Factory that wants to log now must declare its own Logger.
        Chris Male created issue -

          People

          • Assignee:
            Chris Male
            Reporter:
            Chris Male
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development