xiedeyantu commented on code in PR #4656:
URL: https://github.com/apache/calcite/pull/4656#discussion_r2573557899
##########
core/src/main/java/org/apache/calcite/sql/dialect/SqliteSqlDialect.java:
##########
@@ -40,9 +41,30 @@ public class SqliteSqlDialect extends SqlDialect {
public static final SqlDialect DEFAULT = new
SqliteSqlDialect(DEFAULT_CONTEXT);
+ private final int majorVersion;
+ private final int minorVersion;
+
/** Creates a SqliteSqlDialect. */
public SqliteSqlDialect(SqlDialect.Context context) {
super(context);
+ this.majorVersion = context.databaseMajorVersion();
+ this.minorVersion = context.databaseMinorVersion();
+ }
+
+ @Override public boolean supportsJoinType(JoinRelType joinType) {
+ // Unknown version means we conservatively assume support for all join
types
Review Comment:
Done! Thanks! I believe it should be rejected when the version is unknown.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
