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
Generate suggested DDL for HD IMaps without indexes and TS IMaps [HZ-2190] [HZ-2191] #24054
Generate suggested DDL for HD IMaps without indexes and TS IMaps [HZ-2190] [HZ-2191] #24054
Conversation
hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java
Outdated
Show resolved
Hide resolved
The job Click to expand the log file-------------------------- ---------SUMMARY---------- -------------------------- [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.2.1:checkstyle (default) on project hazelcast-sql: An error has occurred in Checkstyle report generation. Failed during checkstyle execution: There is 1 error reported by Checkstyle 8.38 with /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-builder/checkstyle/checkstyle.xml ruleset. -> [Help 1] -------------------------- ---------ERRORS----------- -------------------------- [ERROR] /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-builder/hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java:99:5: Class GetAnyMetadataOperation should be declared as final. [FinalClass] -------------------------- |
The job Click to expand the log file-------------------------- ---------SUMMARY---------- -------------------------- [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.2.1:checkstyle (default) on project hazelcast-sql: An error has occurred in Checkstyle report generation. Failed during checkstyle execution: There is 1 error reported by Checkstyle 8.38 with /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-compiler/checkstyle/checkstyle.xml ruleset. -> [Help 1] -------------------------- ---------ERRORS----------- -------------------------- [ERROR] /home/jenkins/jenkins_slave/workspace/Hazelcast-pr-compiler/hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java:99:5: Class GetAnyMetadataOperation should be declared as final. [FinalClass] -------------------------- |
hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java
Outdated
Show resolved
Hide resolved
The job Click to expand the log file-------------------------- -------TEST FAILURE------- -------------------------- [INFO] Results: [INFO] [ERROR] Failures: [ERROR] MongoStreamSqlConnectorTest.readsFromMongo_withId_oneStep:52->readsFromMongo:110->SqlTestSupport.assertRowsEventuallyInAnyOrder:289 Expecting actual: [Row{[Anakin, Skywalker, insert]}] to contain exactly in any order: [Row{[Luke, Skywalker, insert]}, Row{[Anakin, Skywalker, insert]}] but could not find the following elements: [Row{[Luke, Skywalker, insert]}] |
hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java
Outdated
Show resolved
Hide resolved
hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java
Outdated
Show resolved
Hide resolved
hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java
Show resolved
Hide resolved
hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java
Show resolved
Hide resolved
# Conflicts: # hazelcast-sql/src/test/java/com/hazelcast/jet/sql/MCMessageTasksTest.java
run-lab-run |
The job Click to expand the log file-------------------------- -------TEST FAILURE------- -------------------------- [INFO] Results: [INFO] [ERROR] Failures: [ERROR] MongoStreamSqlConnectorTest.readsFromMongo_withoutId_twoSteps:57->readsFromMongo:111->SqlTestSupport.assertRowsEventuallyInAnyOrder:293 Expecting actual: [Row{[Anakin, Skywalker, insert]}] to contain exactly in any order: [Row{[Luke, Skywalker, insert]}, Row{[Anakin, Skywalker, insert]}] but could not find the following elements: [Row{[Luke, Skywalker, insert]}] |
hazelcast-sql/src/main/java/com/hazelcast/jet/sql/impl/connector/map/MetadataResolver.java
Show resolved
Hide resolved
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 comment for a nice to have thing + LGTM
Removed leftover checks from time when iteration on HD maps required index. Added
beforeOperation()
required by Tiered Storage.Fixes HZ-2190, HZ-2191
EE PR: https://github.com/hazelcast/hazelcast-enterprise/pull/5842
Checklist:
Team:
,Type:
,Source:
,Module:
) and Milestone setAdd to Release Notes
orNot Release Notes content
set