[DATAFU-42] Simplify BagGroup output

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Assignee: Matthew Hayes
• Reporter: Sam Steingold
• Created: 28/Apr/14 14:41
• Updated: 18/May/14 21:23
• Resolved: 16/May/14 19:54

Description

BagGroup keeps the redundant group information in its output.
E.g., see DATAFU-38:

(1,{(b,1),(a,2)},{(B,{(B,3)}),(A,{(A,1),(A,2)})})
(2,{(c,1),(b,2)},{(B,{(B,3),(B,5)}),(A,{(A,1),(A,2)}),(C,{(C,4),(C,6)})})

can be

(1,{(b,1),(a,2)},{(B,{3}),(A,{1,2})})
(2,{(c,1),(b,2)},{(B,{3,5}),(A,{1,2}),(C,{4,6})})

without loss of information

Given that the bug DATAFU-38 rendered this function quite useless and it was fixed just last week, I think BagGroup has not been used before, so this backward-incompatible change will not break any existing code.