Elasticsearch: Missing Keyed Response for Percentiles Bucket

[gingerwizard]

Elasticsearch version:
5.1

I would expect the Percentiles Bucket pipeline aggregation to support a keyed response, similar to the main percentiles agg. Attempting to do this throws the error below, indicating its not currently supported.

{
  "error": {
    "root_cause": [
      {
        "type": "parsing_exception",
        "reason": "Unexpected token VALUE_BOOLEAN [keyed] in [lower_bound]",
        "line": 32,
        "col": 17
      }
    ],
    "type": "parsing_exception",
    "reason": "Unexpected token VALUE_BOOLEAN [keyed] in [lower_bound]",
    "line": 32,
    "col": 17
  },
  "status": 400
}

This introduces a few challenges - primarily the fields produced contains dots with are not accessible in templates.

[polyfractal]

@elastic/es-search-aggs

[gauravmishra123]

Hi I am a newbie , Can I take this issue if not taken ?
Any pointers would be appreciated.

[mayya-sharipova]

@gauravmishra123 This issue is not taken, so please work on this if you like.
You can start with working PercentilesBucketPipelineAggregationBuilder.java file to make a parser accept keyed field. See how the parser in PercentilesAggregationBuilder accepts keyed field

[gauravmishra123]

Thanks a ton @mayya-sharipova for the help.
I have started working on this.

[gauravmishra123]

Hi

I will be committing the fix this week only

On Wed, Jun 27, 2018, 10:57 AM Caleb Ju [e-mail] wrote:

Hi @mayya-sharipova https://github.com/mayya-sharipova , I am also
looking to commit for the first time :). It doesn't look like
@gauravmishra123 https://github.com/gauravmishra123 has committed a fix
yet.

Do you know how I can go testing the
PercentilesBucketPipelineAggregationBuilder.java to see how the parser
interacts with it?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/elastic/elasticsearch/issues/22302#issuecomment-400546852,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZMD7pxXW6-EtHNblJYbtFmHhOGKZCvfks5uAxfVgaJpZM4LS3gz
.

[jucaleb4]

@gauravmishra123 good job! Thanks for letting me know

[not-napoleon]

Hi @gauravmishra123 , I was thinking about picking up this issue since it seems you haven't had a chance to get to it. If you're still working it, please let me know. Thanks!

[gauravmishra123]

Hi @not-napoleon I am working on this.
Since it is my fist opent source , it took me quite some time to understand the code.
I am trying my best to commit it this month end or first week of september.

[not-napoleon]

@gauravmishra123 sounds good. If you get stuck or have a question, feel free to leave a note here and I'll do my best to help.

[joaofcmb]

Hi @gauravmishra123 , I would like to pick up this issue. Are you still working on it?

[gauravmishra123]

No @joaofcmb , I had started to work on this but has got some situtation at my home ,so couldn't get time .You can proceed on this.