Skip to content

Conversation

@ben5556
Copy link
Contributor

@ben5556 ben5556 commented Sep 17, 2018

Corrected aggregation name from intraday_return to hat_prices to match the aggregation name provided in the example

Corrected aggregation name from intraday_return to hat_prices to match the aggregation name provided in the example
@lcawl lcawl added the :Analytics/Aggregations Aggregations label Sep 17, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

LGTM

@lcawl lcawl merged commit 4b57a57 into elastic:6.2 Sep 18, 2018
lcawl pushed a commit to lcawl/elasticsearch that referenced this pull request Sep 18, 2018
@ben5556 ben5556 deleted the patch-1 branch September 18, 2018 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants