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
[CALCITE-5593] Support aliases query in Elasticsearch adapters #3120
Conversation
elasticsearch/src/main/java/org/apache/calcite/adapter/elasticsearch/ElasticsearchSchema.java
Show resolved
Hide resolved
547e260
to
311c41d
Compare
The commit message will become the release notes. It needs to describe the feature. Currently it doesn't mention that it concerns the Elasticsearch adatper. |
311c41d
to
ced719f
Compare
thanks for the reminder, I have fixed the commit message |
@@ -189,6 +191,11 @@ private CalciteAssert.AssertThat calciteAssert() { | |||
.returnsCount(0); | |||
} | |||
|
|||
@Test void testAlias() { | |||
calciteAssert() | |||
.query("select state, city from elastic.zips_alias"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test seems asserting nothing, hence it won't be run actually.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
6a94a07
to
ac3b743
Compare
@coolbeevip The CI failed now, it seems that you are using an guava feature which is not available in our oldest supported guava version: 16.0.1 |
Thanks for the tip, I will use Java StreamSupport.stream instead Guava Streams.stream |
ac3b743
to
8b9d164
Compare
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, merging. @coolbeevip Thanks for your contribution.
Jira issue https://issues.apache.org/jira/browse/CALCITE-5593