Uploaded image for project: 'Samza'
  1. Samza
  2. SAMZA-1347

GroupByContainerIds NPE if containerIds list is null

    Details

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

      Description

      Another low-hanging fruit. The null check in group() happens AFTER checking the size of containerIds. These statements should be reordered.

      Also, Since the group(tasks) method always calls the group(tasks, containerIds) method, lets just move all the safety checks there.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                jmakes Jake Maes
                Reporter:
                jmakes Jake Maes
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: