Use Key API in ResultSet

Summary:
Depends on D14099

Notes:

- Tests pass without many modifications outside of resultset.cpp/.h
- `mGenerator` doesn't seem to be used?

Benchmarks
=========

Run benchmarks:

| Develop                            | D14099                             | This patch                         |
| ---------------------------------- | ---------------------------------- | ---------------------------------- |
| Current Rss usage [kb]: 40700      | Current Rss usage [kb]: 38564      | Current Rss usage [kb]: 39112      |
| Peak Rss usage [kb]: 40700         | Peak Rss usage [kb]: 38564         | Peak Rss usage [kb]: 39112         |
| Rss growth [kb]: 15920             | Rss growth [kb]: 13352             | Rss growth [kb]: 13432             |
| Rss growth per entity [byte]: 3260 | Rss growth per entity [byte]: 2734 | Rss growth per entity [byte]: 2750 |
| Rss without db [kb]: 29736         | Rss without db [kb]: 29248         | Rss without db [kb]: 30100         |
| Percentage peak rss error: 0       | Percentage peak rss error: 0       | Percentage peak rss error: 0       |
| On disk [kb]: 10788                | On disk [kb]: 9140                 | On disk [kb]: 8836                 |
| Buffer size total [kb]: 898        | Buffer size total [kb]: 898        | Buffer size total [kb]: 898        |
| Write amplification: 12.0075       | Write amplification: 10.1732       | Write amplification: 9.83485       |

Test Disk Usage:

| Develop                             | D14099                              | This patch                          |
| ----------------------------------- | ----------------------------------- | ----------------------------------- |
| Free pages: 412                     | Free pages: 309                     | Free pages: 312                     |
| Total pages: 760                    | Total pages: 599                    | Total pages: 603                    |
| Used size: 1425408                  | Used size: 1187840                  | Used size: 1191936                  |
| Calculated key + value size: 856932 | Calculated key + value size: 702866 | Calculated key + value size: 702866 |
| Calculated total db sizes: 970752   | Calculated total db sizes: 954368   | Calculated total db sizes: 933888   |
| Main store on disk: 3112960         | Main store on disk: 2453504         | Main store on disk: 2469888         |
| Total on disk: 3293184              | Total on disk: 2633728              | Total on disk: 2650112              |
| Used size amplification: 1.66339    | Used size amplification: 1.68999    | Used size amplification: 1.69582    |
| Write amplification: 3.63268        | Write amplification: 3.49071        | Write amplification: 3.51402        |

Reviewers: cmollekopf

Reviewed By: cmollekopf

Tags: #sink

Differential Revision: https://phabricator.kde.org/D14289

1 files changed, 13 insertions, 8 deletions

diff --git a/common/resultset.h b/common/resultset.h
index 5587c54..9d83c67 100644
--- a/common/resultset.h
+++ b/common/resultset.h
@@ -25,6 +25,7 @@
 #include "metadata_generated.h"
 #include "entitybuffer.h"
 #include "applicationdomaintype.h"
+#include "storage/key.h"
 
 /*
  * An iterator to a result set.
@@ -35,21 +36,25 @@ class ResultSet
 {
 public:
     struct Result {
-        Result(const Sink::ApplicationDomain::ApplicationDomainType &e, Sink::Operation op, const QMap<QByteArray, QVariant> &v = {}, const QVector<QByteArray> &a = {}) : entity(e), operation(op), aggregateValues(v), aggregateIds(a) {}
+        Result(const Sink::ApplicationDomain::ApplicationDomainType &e, Sink::Operation op,
+            const QMap<QByteArray, QVariant> &v = {}, const QVector<Sink::Storage::Identifier> &a = {})
+            : entity(e), operation(op), aggregateValues(v), aggregateIds(a)
+        {
+        }
         Sink::ApplicationDomain::ApplicationDomainType entity;
         Sink::Operation operation;
         QMap<QByteArray, QVariant> aggregateValues;
-        QVector<QByteArray> aggregateIds;
+        QVector<Sink::Storage::Identifier> aggregateIds;
     };
     typedef std::function<void(const Result &)> Callback;
     typedef std::function<bool(Callback)> ValueGenerator;
-    typedef std::function<QByteArray()> IdGenerator;
+    typedef std::function<Sink::Storage::Identifier()> IdGenerator;
     typedef std::function<void()> SkipValue;
 
     ResultSet();
     ResultSet(const ValueGenerator &generator, const SkipValue &skip);
     ResultSet(const IdGenerator &generator);
-    ResultSet(const QVector<QByteArray> &resultSet);
+    ResultSet(const QVector<Sink::Storage::Identifier> &resultSet);
     ResultSet(const ResultSet &other);
 
     bool next();
@@ -63,14 +68,14 @@ public:
     };
     ReplayResult replaySet(int offset, int batchSize, const Callback &callback);
 
-    QByteArray id();
+    Sink::Storage::Identifier id();
 
     bool isEmpty();
 
 private:
-    QVector<QByteArray> mResultSet;
-    QVector<QByteArray>::ConstIterator mIt;
-    QByteArray mCurrentValue;
+    QVector<Sink::Storage::Identifier> mResultSet;
+    QVector<Sink::Storage::Identifier>::ConstIterator mIt;
+    Sink::Storage::Identifier mCurrentValue;
     IdGenerator mGenerator;
     ValueGenerator mValueGenerator;
     SkipValue mSkip;