fudianchn commented on PR #39613: URL: https://github.com/apache/shardingsphere/pull/39613#issuecomment-5460183020
@terrymanu The E2E failures came from the count guard firing on legitimate rewrites: sharding rewrites AVG to SUM and COUNT and appends order-by and group-by columns, so the backend column count differs from the expanded projection count, and the guard fell back to backend metadata for those queries. The count check now applies only when the rewrite cannot change the column count (no aggregation projection and no derived order-by or group-by projection in the projections context); aggregate and derived-column queries keep the projection-derived metadata as before. Unit tests for these shapes were added in ShardingSphereResultSetUtilsTest, ShardingSphereResultSetMetaDataTest and MySQLProjectionMetadataResolverTest, and they fail with the previous guard. -- 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]
