Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HistogramFieldMapper typo fixed and test fixed #108804

Merged
merged 3 commits into from
Jun 6, 2024

Conversation

snurf198
Copy link
Contributor

Fix typo in HistogramFieldMapper and fixed related test code

closes #108050

@elasticsearchmachine elasticsearchmachine added v8.15.0 needs:triage Requires assignment of a team area label external-contributor Pull request authored by a developer outside the Elasticsearch team labels May 18, 2024
@snurf198 snurf198 changed the title Hfm typo HistogramFieldMapper typo fixed and test fixed May 18, 2024
@gbanasiak gbanasiak added the :Analytics/Aggregations Aggregations label May 22, 2024
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) and removed needs:triage Requires assignment of a team area label labels May 22, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@martijnvg martijnvg added :StorageEngine/Mapping The storage related side of mappings >non-issue and removed :Analytics/Aggregations Aggregations labels Jun 6, 2024
@martijnvg martijnvg self-requested a review June 6, 2024 12:43
@martijnvg martijnvg self-assigned this Jun 6, 2024
@martijnvg
Copy link
Member

@elasticmachine update branch

@elasticsearchmachine elasticsearchmachine added Team:StorageEngine and removed Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Jun 6, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@lkts
Copy link
Contributor

lkts commented Jun 6, 2024

buildkite test this

@martijnvg martijnvg added the auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jun 6, 2024
@elasticsearchmachine elasticsearchmachine merged commit 2822fa4 into elastic:main Jun 6, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue :StorageEngine/Mapping The storage related side of mappings Team:StorageEngine v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typo in DenseVectorFieldMapper.java and HistogramFieldMapper.java
6 participants