Github user keith-turner commented on the issue:
https://github.com/apache/accumulo/pull/180
> Perhaps I didn't do a good job of posing the question explicitly: is it
possible to implement this merge generically? Are there cases which we
could/should support which would require some specialized knowledge or
understanding to merge?
Yesterday I toyed around with changing `merge(Map, Map)` to `merge(String,
long, long)` and this worked beautifully for the `ExampleSummarizer`. Its
merge method became something like the following :
```java
public void merge(String summaryKey, long v1, long v2){
switch(summaryKey){
case "deletes":
case "total":
return v1 + v2;
case "minStamp":
return Math.min(v1, v2);
case "maxStamp":
return Math.max(v1,v2);
}
}
```
Then I went to change `ColumnVisibilitySummarizer` to the new interface and
found it would not work at all. Maybe would could offer a an Abstract subclass
that has an abstract `merge(String, long, long)` method? I might try this just
to see what it looks like.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
