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

[BUG] New engine doesn't support duplicated identifiers in the SELECT clause #1382

Closed
GumpacG opened this issue Feb 24, 2023 · 2 comments
Closed
Labels
bug Something isn't working new-engine-gap untriaged

Comments

@GumpacG
Copy link
Collaborator

GumpacG commented Feb 24, 2023

What is the bug?
New engine doesn't support duplicated identifiers in the SELECT clause. This causes a legacy IT to fail when it runs on the new engine. https://github.com/opensearch-project/sql/blob/main/integ-test/src/test/java/org/opensearch/sql/legacy/QueryIT.java#:~:text=selectAllWithFieldReturnsAll

How can one reproduce the bug?
Steps to reproduce the behavior:

  1. Query SELECT *, int0 from calcs
  2. See error
  "error": {
    "reason": "Invalid SQL query",
    "details": "Multiple entries with same key: int0=1 and int0=1",
    "type": "IllegalArgumentException"
  },
  "status": 400
}

What is the expected behavior?
The output should be similar to Legacy where all columns are included in the result

What is your host/environment?

  • OpenSearch version 2.x
@GumpacG GumpacG added bug Something isn't working untriaged labels Feb 24, 2023
@Yury-Fridlyand
Copy link
Collaborator

A duplicate of #785?

@GumpacG
Copy link
Collaborator Author

GumpacG commented Feb 24, 2023

A duplicate of #785?

Yes, closing this issue. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working new-engine-gap untriaged
Projects
None yet
Development

No branches or pull requests

3 participants