[
https://issues.apache.org/jira/browse/DATAFU-42?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14000307#comment-14000307
]
Matthew Hayes commented on DATAFU-42:
-------------------------------------
Thanks Jarek for committing my change :)
> Simplify BagGroup output
> ------------------------
>
> Key: DATAFU-42
> URL: https://issues.apache.org/jira/browse/DATAFU-42
> Project: DataFu
> Issue Type: Improvement
> Reporter: Sam Steingold
> Assignee: Matthew Hayes
> Fix For: 1.3.0
>
> Attachments: DATAFU-42-v2.patch, DATAFU-42.patch
>
>
> {{BagGroup}} keeps the redundant {{group}} information in its output.
> E.g., see [DATAFU-38]:
> {code}
> (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)})})
> {code}
> can be
> {code}
> (1,{(b,1),(a,2)},{(B,{3}),(A,{1,2})})
> (2,{(c,1),(b,2)},{(B,{3,5}),(A,{1,2}),(C,{4,6})})
> {code}
> 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.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
