Tapestry
  1. Tapestry
  2. TAPESTRY-971

Include a NullDataSqueezerFilter in the data squeezer pipeline

    Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.1
    • Fix Version/s: 4.1
    • Component/s: Framework
    • Labels:
      None

      Description

      Please add a NullDataSqueezerFilter to the front of the data squeezer pipeline. Essentially, this will be a short-circuit implementation that makes sure nulls are squeezed/unsqueezed properly and alleviates the burden of null checks on downstream filters.

      1. patch.txt
        7 kB
        James Carman
      2. patch.txt
        7 kB
        James Carman

        Activity

        James Carman created issue -
        Hide
        James Carman added a comment -

        Here's a patch that implements it.

        Show
        James Carman added a comment - Here's a patch that implements it.
        James Carman made changes -
        Field Original Value New Value
        Attachment patch.txt [ 12334834 ]
        Hide
        James Carman added a comment -

        Fixed ordering.

        Show
        James Carman added a comment - Fixed ordering.
        James Carman made changes -
        Attachment patch.txt [ 12334836 ]
        Jesse Kuhnert made changes -
        Assignee Jesse Kuhnert [ jkuhnert ]
        Hide
        Jesse Kuhnert added a comment -

        Applied.

        Show
        Jesse Kuhnert added a comment - Applied.
        Jesse Kuhnert made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Fix Version/s 4.1 [ 12310632 ]
        Mark Thomas made changes -
        Workflow jira [ 12372636 ] Default workflow, editable Closed status [ 12567767 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12567767 ] jira [ 12590895 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        57d 3h 41m 1 Jesse Kuhnert 28/Jul/06 03:31

          People

          • Assignee:
            Jesse Kuhnert
            Reporter:
            James Carman
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development