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]

Reply via email to