Skip to content

Commit

Permalink
revert change in SegmentMetadataQuery style change
Browse files Browse the repository at this point in the history
  • Loading branch information
cecemei committed Jan 10, 2025
1 parent b19f657 commit a1359a6
Showing 1 changed file with 4 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ public static AnalysisType fromString(String name)
@Override
public byte[] getCacheKey()
{
return new byte[]{(byte) this.ordinal()};
return new byte[] {(byte) this.ordinal()};
}
}

Expand Down Expand Up @@ -116,12 +116,9 @@ public SegmentMetadataQuery(
// of truth for consumers of this class variable. The defaults are to preserve backwards compatibility.
// In a future release, 28.0+, we can remove the deprecated property lenientAggregatorMerge.
if (lenientAggregatorMerge != null && aggregatorMergeStrategy != null) {
throw InvalidInput.exception(
"Both lenientAggregatorMerge [%s] and aggregatorMergeStrategy [%s] parameters cannot be set."
+ " Consider using aggregatorMergeStrategy since lenientAggregatorMerge is deprecated.",
lenientAggregatorMerge,
aggregatorMergeStrategy
);
throw InvalidInput.exception("Both lenientAggregatorMerge [%s] and aggregatorMergeStrategy [%s] parameters cannot be set."
+ " Consider using aggregatorMergeStrategy since lenientAggregatorMerge is deprecated.",
lenientAggregatorMerge, aggregatorMergeStrategy);
}
if (lenientAggregatorMerge != null) {
this.aggregatorMergeStrategy = lenientAggregatorMerge
Expand Down

0 comments on commit a1359a6

Please sign in to comment.