terrymanu opened a new issue, #18995:
URL: https://github.com/apache/shardingsphere/issues/18995
The current structures are:
```
RALStatement
-- QueryableRALStatement
-- UpdatableRALStatement
-- QueryableScalingRALStatement
-- UpdatableScalingRALStatement
-- CommonRALStatement
-- AdvancedDistSQLStatement
```
`QueryableRALStatement` and `UpdatableRALStatement` is the basic
distinguished categories which is suitable.
`QueryableScalingRALStatement` and `UpdatableScalingRALStatement` are not in
same abstract level with previous.
And `CommonRALStatement` is not suitable too for same abstract level and
for hint only.
`AdvancedDistSQLStatement` is not used for now, maybe remove soon.
I suggest to adjust the class extension structure. How about below?
```
RALStatement
-- QueryableRALStatement
-- UpdatableRALStatement
-- ScalingRALStatement
---- QueryableScalingRALStatement
---- UpdatableScalingRALStatement
-- HintRALStatement
```
--
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]