Skip to content

Commit e1279da

Browse files
committed
Removed unused code; clean up error message
Signed-off-by: Asim Mahmood <[email protected]>
1 parent 0fafb49 commit e1279da

File tree

2 files changed

+1
-14
lines changed

2 files changed

+1
-14
lines changed

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregationBuilder.java

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -157,24 +157,11 @@ public CardinalityAggregationBuilder precisionThreshold(long precisionThreshold)
157157
return this;
158158
}
159159

160-
/**
161-
* Get the precision threshold. Higher values improve accuracy but also
162-
* increase memory usage. Will return <code>null</code> if the
163-
* precisionThreshold has not been set yet.
164-
*/
165-
public Long precisionThreshold() {
166-
return precisionThreshold;
167-
}
168-
169160
public CardinalityAggregationBuilder executionHint(String executionHint) {
170161
this.executionHint = executionHint;
171162
return this;
172163
}
173164

174-
public String executionHint() {
175-
return executionHint;
176-
}
177-
178165
@Override
179166
protected CardinalityAggregatorFactory innerBuild(
180167
QueryShardContext queryShardContext,

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregatorFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ public static ExecutionMode fromString(String value) {
6969
try {
7070
return ExecutionMode.valueOf(value.toUpperCase(Locale.ROOT));
7171
} catch (IllegalArgumentException e) {
72-
throw new IllegalArgumentException("Unknown `execution_hint`: [" + value + "], expected any of [direct, ordinals]");
72+
throw new IllegalArgumentException("Unknown execution_hint: [" + value + "], expected any of [direct, ordinals]");
7373
}
7474
}
7575

0 commit comments

Comments
 (0)