database/mariadb-columnstore-engine
1
0
Fork 0
mirror of https://github.com/mariadb-corporation/mariadb-columnstore-engine.git synced 2025-07-30 19:23:07 +03:00
Code Activity

fix(cmapi): MCOL-5133 remove has_active_nodes handler for cluster shutdown because TransactionManager has advance variant of this check

Browse Source
This commit is contained in:
mariadb-AlanMologorsky
2025-03-20 15:14:40 +03:00
committed by Leonid Fedorov
parent 0e5842abcc
commit 2f25c47a57
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
cmapi/cmapi_server/controllers/endpoints.py
View File

@ -157,7 +157,13 @@ def active_operation():
@cherrypy.tools.register('before_handler', priority=82) @cherrypy.tools.register('before_handler', priority=82)
def has_active_nodes(): def has_active_nodes():
"""Check if there are any active nodes in the cluster.""" """Check if there are any active nodes in the cluster.
TODO: Remove in next releases due to never used.
Now TransactionManager has this check inside.
Before removing, have to check all API endpoints without transaction
mechanics to potential use of this handler.
"""
active_nodes = get_active_nodes() active_nodes = get_active_nodes()
if len(active_nodes) == 0: if len(active_nodes) == 0:
@ -834,7 +840,6 @@ class ClusterController:
@cherrypy.tools.json_in() @cherrypy.tools.json_in()
@cherrypy.tools.json_out() @cherrypy.tools.json_out()
@cherrypy.tools.validate_api_key() # pylint: disable=no-member @cherrypy.tools.validate_api_key() # pylint: disable=no-member
@cherrypy.tools.has_active_nodes() # pylint: disable=no-member
def put_shutdown(self): def put_shutdown(self):
func_name = 'put_shutdown' func_name = 'put_shutdown'
log_begin(module_logger, func_name) log_begin(module_logger, func_name)