Pig
  1. Pig
  2. PIG-2065

IsEmpty should be Accumulative

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      IsEmpty is trivial to implement as an accumulator. We should do that.

        Issue Links

          Activity

          Hide
          Jonathan Coveney added a comment -

          Why is this necessary? At least for DefaultDataBags, the size is simply a known quantity, so you can check if "size == 0." Do you have another case in mind?

          Show
          Jonathan Coveney added a comment - Why is this necessary? At least for DefaultDataBags, the size is simply a known quantity, so you can check if "size == 0." Do you have another case in mind?
          Hide
          Thejas M Nair added a comment -

          Why is this necessary?

          The plan will change if isEmpty implements accumulator. The entire (potentially large) bags would not have to be loaded into memory.

          Show
          Thejas M Nair added a comment - Why is this necessary? The plan will change if isEmpty implements accumulator. The entire (potentially large) bags would not have to be loaded into memory.
          Hide
          Jonathan Coveney added a comment -

          This has already been implemented as part of another patch.

          Show
          Jonathan Coveney added a comment - This has already been implemented as part of another patch.

            People

            • Assignee:
              Jonathan Coveney
              Reporter:
              Dmitriy V. Ryaboy
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development