Keep private access.

2 files changed, 3 insertions, 2 deletions

diff --git a/searchlib/src/tests/grouping/grouping_test.cpp b/searchlib/src/tests/grouping/grouping_test.cpp
index d1430322c26..94d3afd619b 100644
--- a/searchlib/src/tests/grouping/grouping_test.cpp
+++ b/searchlib/src/tests/grouping/grouping_test.cpp
@@ -307,7 +307,6 @@ void
 Test::testAggregationSimple()
 {
     EXPECT_EQUAL(64u, sizeof(Group));
-    EXPECT_EQUAL(40u, sizeof(Group::Value));
     AggregationContext ctx;
     ctx.result().add(0).add(1).add(2);
     ctx.add(IntAttrBuilder("int").add(3).add(7).add(15).sp());
diff --git a/searchlib/src/vespa/searchlib/aggregation/group.h b/searchlib/src/vespa/searchlib/aggregation/group.h
index 2c0e72b543b..f6b6bc732af 100644
--- a/searchlib/src/vespa/searchlib/aggregation/group.h
+++ b/searchlib/src/vespa/searchlib/aggregation/group.h
@@ -58,6 +58,8 @@ public:
 
     using GroupingLevelList = std::vector<GroupingLevel>;
 
+private:
+
     class Value {
     public:
         Value();
@@ -152,7 +154,7 @@ public:
         uint8_t          _packedLength;    // Length of aggr and expr vectors.
         uint8_t          _orderBy[2];           // How this group is ranked, negative means reverse rank.
     };
-private:
+
     ResultNode::CP   _id;                   // the label of this group, separating it from other groups
     RawRank          _rank;                 // The default rank taken from the highest hit relevance.
     Value            _aggr;