tuichenchuxin commented on PR #20311: URL: https://github.com/apache/shardingsphere/pull/20311#issuecomment-1222213623
> > Looks Greate! Can you briefly the design principles? @tuichenchuxin > > > `KILL <processID>` is depends on the improvement of `show processlist` , kill the processId of the process list. > > design principles: We don't know the processID belongs to which `shardingSphere` instance, so when `shardingSphere` receives the Kill Command, it needs to notify each node, each node determines whether it currently has this ProcessId. If yes, find the statement connecting to the database according to the ProcessId, and then call statement.cancel to cancel the SQL task. > > The details are as follows: > > 1. DDLStatement and DMLStatement need save processID to Statement Map > 2. Use the `zookeeper` node to implement wait for notification mechanism Got it. LGTM. This pr is only for cluster? -- 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]
