Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.5, Trunk
    • Component/s: core/search
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      There are a couple of use cases I can think of where being able to get the underlying filter out of CachingWrapperFilter would be useful:
      1. You might want to introspect the filter to figure out what's in it (the use case we hit.)
      2. You might want to serialise the filter since Lucene no longer supports that itself.

      We currently work around this by subclassing, keeping another copy of the underlying filter reference and implementing a trivial getter, which is an easy workaround, but the trap is that a junior developer could unknowingly create a CachingWrapperFilter without knowing that the BetterCachingWrapperFilter exists, introducing a filter which cannot be introspected.

        Activity

        Adrien Grand made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Adrien Grand made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 5.0 [ 12321663 ]
        Fix Version/s 4.5 [ 12324742 ]
        Resolution Fixed [ 1 ]
        Trejkaz made changes -
        Attachment LUCENE-5186.patch [ 12599572 ]
        Adrien Grand made changes -
        Field Original Value New Value
        Assignee Adrien Grand [ jpountz ]
        Trejkaz created issue -

          People

          • Assignee:
            Adrien Grand
            Reporter:
            Trejkaz
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development