Copilot commented on code in PR #35497:
URL: https://github.com/apache/shardingsphere/pull/35497#discussion_r2104841976
##########
infra/binder/core/src/main/java/org/apache/shardingsphere/infra/binder/context/statement/dml/SelectStatementContext.java:
##########
@@ -116,14 +116,14 @@ public final class SelectStatementContext extends
CommonSQLStatementContext impl
private PaginationContext paginationContext;
- public SelectStatementContext(final ShardingSphereMetaData metaData, final
DatabaseType databaseType, final List<Object> params, final SelectStatement
sqlStatement,
- final String currentDatabaseName, final
Collection<TableSegment> inheritedTables) {
+ public SelectStatementContext(final DatabaseType databaseType, final
SelectStatement sqlStatement,
Review Comment:
Consider updating the constructor JavaDoc (or related documentation) to
reflect the new parameter order for clarity.
--
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: notifications-unsubscr...@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
