Gaffer icon indicating copy to clipboard operation
Gaffer copied to clipboard

RowIDAggregator seems to assume column family is the group

Open gaffer01 opened this issue 6 years ago • 1 comments

The RowIdAggregator seems to assume that the column family specifies the group, e.g.

 group = options.get(AccumuloStoreConstants.COLUMN_FAMILY);

and in IteratorSettingFactory the getRowIDAggregatorIteratorSetting method takes parameters final AccumuloStore store, final String columnFamily.

It's not the case that every key package has to have the column family set to the group.

We should check whether this iterator is assuming that the group is the CF and if so fix it.

gaffer01 avatar Mar 05 '18 15:03 gaffer01

This seems to be a bigger ticket than originally thought. We should use this to update the KeyPackages, possibly using a new field to specify where the group is stored. We should then use this within the methods in the ElementConverter to get the groups, therefore not relying on the group to always be equal to the ColumnFamily.

m55624 avatar Apr 10 '18 14:04 m55624