Extend allowed characters in aggregation name

[colings86]

Currently aggregations names must match the pattern [a-zA-Z0-9\\-_]+. This is pretty restrictive and prevents, for example, a common naming pattern of using dots in names. We should extend this pattern to be less restrictive.

Currently an aggregation can refer to values within child aggregations using two methods:

1. agg_name['val_name']
2. agg_name.val_name

Allowing dots in the aggregation name will mean that we drop support for option 2 in favour of 1.

[felixbarny]

Not understaning the reason why only alpa-numeric chars are allowed, wouldn't it be possible to allow any char by internally working with hexadecimal encoded strings? I would really wish, that arbitrary chars would still be allowed. My current workaround is to do the hex encoding manually. My aggregation names are computed dynamically and are displayed on the frontend, so avoiding non-alpha-numerics is not an option for me.

("cross-posted" from #5253)

[clintongormley]

@colings86 What about allowing escaping of dots? eg "foo.bar.baz" (as JSON "foo\.bar.baz")?

[colings86]

relates to #6736

[felixbarny]

Good job!
Will it be included in 1.2.2? Is there some place that shows when the next release is due?

[colings86]

This will be available from version 1.3 as 1.2.2 is intended for bug fixes.

We try to release every couple of months, but it depends on the features we are working on. We don't announce release dates ahead of time.

[gulcabuk]

Hi,

I'm trying to use bucket selection with over an aggregation whose name contains numerical characters. Same error has been occurred for me. I realized that aggregations with numeric characters has been solved but I think there is still an error for bucket selection.

Is there a way to solve this?

Error:

{
  "error" : {
    "root_cause" : [
      {
        "type" : "parsing_exception",
        "reason" : "Invalid aggregation name [secondagg_sum_[filters_equals_100]]. Aggregation names must be alpha-numeric and can only contain '_' and '-'",
        "line" : 1,
        "col" : 318
      }
    ],
    "type" : "parsing_exception",
    "reason" : "Invalid aggregation name [secondagg_sum_[filters_equals_100]]. Aggregation names must be alpha-numeric and can only contain '_' and '-'",
    "line" : 1,
    "col" : 318
  },
  "status" : 400
}

My query:

{
  "size": 0,
  "query": {
    "bool": {
      "disable_coord": false,
      "adjust_pure_negative": true,
      "boost": 1.0
    }
  },
  "aggregations": {
    "first_agg": {
      "terms": {
        "field": "firstproperty",
        "size": 2147483647,
        "min_doc_count": 1,
        "shard_min_doc_count": 0,
        "show_term_doc_count_error": false,
        "order": {
          "_term": "asc"
        }
      },
      "aggregations": {
        "secondagg_sum_[filters_equals_100]": {
          "sum": {
            "field": "secondproperty"
          }
        },
        "agg_values": {
          "bucket_selector": {
            "buckets_path": {
              "total": "secondagg_sum_[filters_equals_100]"
            },
            "script": {
              "inline": "params.total > 100",
              "lang": "painless"
            },
            "gap_policy": "skip"
          }
        }
      }
    }
  }
}

[rjernst]

@gulcabuk Please ask questions on our forum. This issue was closed 3 years ago, and we use github for feature requests and confirmed bug reports.