Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Added support for PERCENTILES in JDBC driver #146

Merged
merged 4 commits into from
Aug 13, 2019
Merged

Added support for PERCENTILES in JDBC driver #146

merged 4 commits into from
Aug 13, 2019

Conversation

galkk
Copy link
Contributor

@galkk galkk commented Aug 9, 2019

Adds PERCENTILE output to JDBC

GET /_opendistro/_sql?format=jdbc
{
  "query": "SELECT percentiles(age, 25.0, 50.0, 75.0, 99.9) age_percentiles FROM peoples"
}

{
  "schema": [{
    "name": "age_percentiles",
    "type": "double"
  }],
  "total": 1,
  "datarows": [[{
    "75.0": 36.5,
    "99.9": 39,
    "25.0": 31.5,
    "50.0": 33.5
  }]],
  "size": 1,
  "status": 200
}

Added new unchecked exception that enables us to say that we clearly know that feature is not implemented (differentiating it from not supported)

Fixes #26

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Added new unchecked exception that enables us to say that we clearly know that feature is not implemented (differentiating it from not supported)

Fixes #37
Added new unchecked exception that enables us to say that we clearly know that feature is not implemented (differentiating it from not supported)

Fixes #27
Added new unchecked exception that enables us to say that we clearly know that feature is not implemented (differentiating it from not supported)

Fixes #26
Copy link
Contributor

@arsen-es arsen-es left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Added new unchecked exception that enables us to say that we clearly know that feature is not implemented (differentiating it from not supported)

Fixes #26
@galkk galkk merged commit 557fdc0 into opendistro-for-elasticsearch:master Aug 13, 2019
@galkk galkk deleted the percentile-support branch August 13, 2019 00:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PERCENTILES aggregation does not work with JDBC format.
4 participants