RaigorJiang opened a new issue, #23772: URL: https://github.com/apache/shardingsphere/issues/23772
Now `DistSQLResultSet` is used to implement RQL functionality, and its implementations contain business logic, which is not good, `ResultSet` should just be an object to collect results. Refer to #23735, #23737 and #23766, we are using `MergedResult` for RQL and removing `DistSQLResultSet` to make the logic clearer. There are many classes to be refactored, if you want, please participate together. ### Tasks - [ ] CountShardingRuleResultSet - [ ] UnusedShardingAlgorithmsResultSet - [ ] ShardingTableRulesUsedAlgorithmResultSet - [ ] CountMaskRuleResultSet - [ ] ShardingTableRulesUsedKeyGeneratorResultSet - [ ] ShowMigrationSourceStorageUnitsResultSet - [ ] RulesUsedStorageUnitResultSet - [ ] ShardingTableReferenceRuleResultSet - [ ] DefaultShardingStrategyResultSet - [ ] CountSingleTableResultSet - [ ] CountShadowRuleResultSet - [ ] ShowMigrationCheckAlgorithmsResultSet - [ ] CountReadwriteSplittingRuleResultSet - [ ] ShadowRuleResultSet - [ ] ShowMigrationCheckStatusResultSet - [ ] LogicalTableResultSet - [ ] DatabaseDiscoveryRuleResultSet - [ ] ShadowTableRuleResultSet - [ ] CountDatabaseDiscoveryRuleResultSet - [ ] ShowMigrationJobStatusResultSet - [ ] ShardingTableRulesUsedAuditorResultSet - [ ] DefaultShadowAlgorithmResultSet - [ ] MaskRuleResultSet - [ ] UnusedShardingAuditorsResultSet - [ ] BroadcastTableRuleResultSet - [ ] CountEncryptRuleResultSet - [ ] ShardingAuditorsResultSet - [ ] ShardingKeyGeneratorResultSet - [ ] ShardingTableNodesResultSet - [ ] UnusedShardingKeyGeneratorResultSet - [ ] ShadowAlgorithmResultSet - [ ] ShowMigrationListResultSet - [ ] ShowMigrationRuleResultSet - [ ] ShardingTableRuleResultSet - [ ] DatabaseDiscoveryTypeResultSet - [ ] DatabaseDiscoveryHeartbeatResultSet - [ ] EncryptRuleResultSet - [ ] ShardingAlgorithmResultSet - [ ] ReadwriteSplittingRuleResultSet -- 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]
