[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Keith Turner (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018322#comment-15018322 ] Keith Turner commented on ACCUMULO-4062: I think the expense should be quantified through a

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Keith Turner (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018312#comment-15018312 ] Keith Turner commented on ACCUMULO-4062: I am uncertain the batch writer provides this

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018153#comment-15018153 ] Dave Marion commented on ACCUMULO-4062: --- Looking into this further - I looked at the server side

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018446#comment-15018446 ] Dave Marion commented on ACCUMULO-4062: --- That's a good point. I assume that you mean when the

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Keith Turner (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018341#comment-15018341 ] Keith Turner commented on ACCUMULO-4062: Personally, I think the issue of deduping the same

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Keith Turner (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018464#comment-15018464 ] Keith Turner commented on ACCUMULO-4062: Remove it or put the combiner below it. May still

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Christopher Tubbs (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018410#comment-15018410 ] Christopher Tubbs commented on ACCUMULO-4062: - This could break combiners and a user's

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Christopher Tubbs (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018455#comment-15018455 ] Christopher Tubbs commented on ACCUMULO-4062: - It's normal to remove the versioning

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-20 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15018753#comment-15018753 ] Josh Elser commented on ACCUMULO-4062: -- Yay, this was a nice discussion :) Thanks for bringing it

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014239#comment-15014239 ] Josh Elser commented on ACCUMULO-4062: -- See the implementation for {{Mutation#hashCode()}}, which

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014213#comment-15014213 ] Dave Marion commented on ACCUMULO-4062: --- Is there a reason not to do this? What am I missing? >

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014259#comment-15014259 ] Dave Marion commented on ACCUMULO-4062: ---

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014276#comment-15014276 ] Dave Marion commented on ACCUMULO-4062: --- Looks like it was done 4 weeks ago in the 1.6 branch and

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014279#comment-15014279 ] Josh Elser commented on ACCUMULO-4062: -- Aha, I am behind! ACCUMULO-4029 > Change

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014268#comment-15014268 ] Josh Elser commented on ACCUMULO-4062: -- Oh, that must be new. The above I copied is in 1.7. That

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014307#comment-15014307 ] Dave Marion commented on ACCUMULO-4062: --- I think your comment about different ordering of updates

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014321#comment-15014321 ] Josh Elser commented on ACCUMULO-4062: -- As far as I understand it, I believe that's the case.

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014337#comment-15014337 ] Dave Marion commented on ACCUMULO-4062: --- I don't think it renders this optimization useless

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread John Vines (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014507#comment-15014507 ] John Vines commented on ACCUMULO-4062: -- Given the strict tracking we have about order of mutations

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread John Vines (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014433#comment-15014433 ] John Vines commented on ACCUMULO-4062: -- Wouldn't this cause writes to arrive within a single

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014466#comment-15014466 ] Dave Marion commented on ACCUMULO-4062: --- I dont think that guarantee ever existed. A client

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Josh Elser (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014477#comment-15014477 ] Josh Elser commented on ACCUMULO-4062: -- bq. I don't think it renders this optimization useless

[jira] [Commented] (ACCUMULO-4062) Change MutationSet.mutations to use HashSet

2015-11-19 Thread Dave Marion (JIRA)
[ https://issues.apache.org/jira/browse/ACCUMULO-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15014550#comment-15014550 ] Dave Marion commented on ACCUMULO-4062: --- So two rows with the exact same key components and