honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix typo in ParsedCardinality comment and add //norelease comment on DocValueFormat dep

This commit is contained in:
javanna 2017-04-14 11:22:44 +02:00 committed by Luca Cavanna
parent 7f730c9489
commit 5ccb4a0bbd
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
core/src/main/java/org/elasticsearch/search/aggregations/metrics/cardinality/ParsedCardinality.java
View File

@ -33,8 +33,9 @@ public class ParsedCardinality extends ParsedAggregation implements Cardinality
@Override @Override
public String getValueAsString() { public String getValueAsString() {
// InternalCardinality doesn't print "value_as_string", but you can get a formated value using // InternalCardinality doesn't print "value_as_string", but you can get a formatted value using
// getValueAsString(). That method uses the raw formatter so we also use it here. // getValueAsString(). That method uses the raw formatter so we also use it here.
//norelease is it worth doing Double.toString(value) and removing the dependency to DocValueFormat.RAW ?
return DocValueFormat.RAW.format((double) cardinalityValue); return DocValueFormat.RAW.format((double) cardinalityValue);
} }