ChangeField: Remove duplicate calls to stored() for int range fields All int range field must be stored. This is why FieldDef#intRange already calls the stored() method. Calling it again for int range fields in ChangeField is unneeded. Change-Id: I04c11c70568a0ac510a9b95bc1a4489a4e34f9e8 Signed-off-by: Edwin Kempin <ekempin@google.com>
diff --git a/java/com/google/gerrit/server/index/change/ChangeField.java b/java/com/google/gerrit/server/index/change/ChangeField.java index 262c732..6dbad6a 100644 --- a/java/com/google/gerrit/server/index/change/ChangeField.java +++ b/java/com/google/gerrit/server/index/change/ChangeField.java
@@ -517,7 +517,6 @@ /** Number of unresolved comments of the change. */ public static final FieldDef<ChangeData, Integer> UNRESOLVED_COMMENT_COUNT = intRange(ChangeQueryBuilder.FIELD_UNRESOLVED_COMMENT_COUNT) - .stored() .build(ChangeData::unresolvedCommentCount); /** Whether the change is mergeable. */ @@ -536,13 +535,11 @@ /** The number of inserted lines in this change. */ public static final FieldDef<ChangeData, Integer> ADDED = intRange(ChangeQueryBuilder.FIELD_ADDED) - .stored() .build(cd -> cd.changedLines().isPresent() ? cd.changedLines().get().insertions : null); /** The number of deleted lines in this change. */ public static final FieldDef<ChangeData, Integer> DELETED = intRange(ChangeQueryBuilder.FIELD_DELETED) - .stored() .build(cd -> cd.changedLines().isPresent() ? cd.changedLines().get().deletions : null); /** The total number of modified lines in this change. */