Skip to content

Commit

Permalink
[ML] use actual field name for agg filtering instead of safeFieldName
Browse files Browse the repository at this point in the history
  • Loading branch information
darnautov committed Jan 21, 2020
1 parent b81d6a1 commit 24f4c05
Showing 1 changed file with 4 additions and 20 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -261,11 +261,7 @@ export class DataVisualizer {
aggregatableFields.forEach((field, i) => {
const safeFieldName = getSafeAggregationName(field, i);
aggs[`${safeFieldName}_count`] = {
filter: {
exists: {
field: safeFieldName,
},
},
filter: { exists: { field } },
};
aggs[`${safeFieldName}_cardinality`] = {
cardinality: { field },
Expand Down Expand Up @@ -437,11 +433,7 @@ export class DataVisualizer {
fields.forEach((field, i) => {
const safeFieldName = getSafeAggregationName(field.fieldName, i);
aggs[`${safeFieldName}_field_stats`] = {
filter: {
exists: {
field: safeFieldName,
},
},
filter: { exists: { field: field.fieldName } },
aggs: {
actual_stats: {
stats: { field: field.fieldName },
Expand Down Expand Up @@ -654,11 +646,7 @@ export class DataVisualizer {
fields.forEach((field, i) => {
const safeFieldName = getSafeAggregationName(field.fieldName, i);
aggs[`${safeFieldName}_field_stats`] = {
filter: {
exists: {
field: safeFieldName,
},
},
filter: { exists: { field: field.fieldName } },
aggs: {
actual_stats: {
stats: { field: field.fieldName },
Expand Down Expand Up @@ -720,11 +708,7 @@ export class DataVisualizer {
fields.forEach((field, i) => {
const safeFieldName = getSafeAggregationName(field.fieldName, i);
aggs[`${safeFieldName}_value_count`] = {
filter: {
exists: {
field: safeFieldName,
},
},
filter: { exists: { field: field.fieldName } },
};
aggs[`${safeFieldName}_values`] = {
terms: {
Expand Down

0 comments on commit 24f4c05

Please sign in to comment.