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.

Sign up for GitHub

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

Jump to bottom

SQL: Make mapping returned from SQL Commands lowercase #37521

Closed
matriv opened this issue Jan 16, 2019 · 1 comment
Closed

SQL: Make mapping returned from SQL Commands lowercase #37521

matriv opened this issue Jan 16, 2019 · 1 comment
Assignees
@matriv
Labels
:Analytics/SQL SQL querying >non-issue

Comments

@matriv
Copy link
Contributor

matriv commented Jan 16, 2019 •
edited
Loading

To follow the ES convention, make mapping lowercase, e.g.:
https://github.com/elastic/elasticsearch/blob/master/x-pack/plugin/sql/qa/src/main/resources/alias.csv-spec#L59
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@matriv matriv mentioned this issue Jan 16, 2019
Merged
@matriv matriv self-assigned this Jan 16, 2019
@matriv matriv changed the title SQL: Make mapping returned in DESCRIBE lowercase SQL: Make mapping returned from SQL Commands lowercase Jan 16, 2019
matriv added a commit to matriv/elasticsearch that referenced this issue Jan 16, 2019
@matriv
SQL: Lowercase es data type (mapping) returned from SQL Commands
90f2468 
To follow the ES convention, convert the es data type, returned as
column `mapping` from SQL Commands, to lowercase.

Fixes: elastic#37521
@matriv matriv mentioned this issue Jan 16, 2019
Merged
matriv added a commit that referenced this issue Jan 16, 2019
@matriv
SQL: Lowercase es data type (mapping) returned from SQL Commands (#37531
2cf4b18 
)

To follow the ES convention, convert the es data type, returned as
column `mapping` from SQL Commands, to lowercase.

Fixes: #37521
matriv added a commit that referenced this issue Jan 16, 2019
@matriv
SQL: Lowercase es data type (mapping) returned from SQL Commands (#37531
9c43365 
)

To follow the ES convention, convert the es data type, returned as
column `mapping` from SQL Commands, to lowercase.

Fixes: #37521
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matriv matriv

Labels
:Analytics/SQL SQL querying >non-issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@matriv @elasticmachine