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

Typo in DenseVectorFieldMapper.java and HistogramFieldMapper.java #108050

Open
snurf198 opened this issue Apr 30, 2024 · 1 comment · Fixed by #108065 · May be fixed by #108804
Open

Typo in DenseVectorFieldMapper.java and HistogramFieldMapper.java #108050

snurf198 opened this issue Apr 30, 2024 · 1 comment · Fixed by #108065 · May be fixed by #108804
Labels
>bug :Search/Mapping Index mappings, including merging and defining field types Team:Search Meta label for search team

Comments

@snurf198
Copy link

snurf198 commented Apr 30, 2024

Elasticsearch Version

8.13.2

Installed Plugins

No response

Java Version

java 17.0.7 2023-04-18 LTS Java(TM) SE Runtime Environment (build 17.0.7+8-LTS-224) Java HotSpot(TM) 64-Bit Server VM (build 17.0.7+8-LTS-224, mixed mode, sharing)

OS Version

Darwin 23.4.0 Darwin Kernel Version 23.4.0: Fri Mar 15 00:11:05 PDT 2024; root:xnu-10063.101.17~1/RELEASE_X86_64 x86_64

Problem Description

in both

./server/src/main/java/org/elasticsearch/index/mapper/vectors/DenseVectorFieldMapper.java:1417
./x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/mapper/HistogramFieldMapper.java:413

files, there is a typo.

+ "] doesn't not support indexing multiple values for the same field in the same document"
should be changed to
+ "] does not support indexing multiple values for the same field in the same document"

also, the corresponding test code should be changed.

Steps to Reproduce

.

Logs (if relevant)

No response

@snurf198 snurf198 added >bug needs:triage Requires assignment of a team area label labels Apr 30, 2024
@pxsalehi pxsalehi added the :Search/Mapping Index mappings, including merging and defining field types label Apr 30, 2024
@elasticsearchmachine elasticsearchmachine added the Team:Search Meta label for search team label Apr 30, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search (Team:Search)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Search/Mapping Index mappings, including merging and defining field types Team:Search Meta label for search team
Projects
None yet
4 participants