add TODO to use a future ucx feature default tip

Wed, 27 Nov 2024 23:00:07 +0100

author
Olaf Wintermann <olaf.wintermann@gmail.com>
date
Wed, 27 Nov 2024 23:00:07 +0100
changeset 563
6ca97c99173e
parent 562
69fc4ecc5f60

add TODO to use a future ucx feature

src/server/daemon/resourcepool.c file | annotate | diff | comparison | revisions
--- a/src/server/daemon/resourcepool.c	Fri Nov 01 12:25:52 2024 +0100
+++ b/src/server/daemon/resourcepool.c	Wed Nov 27 23:00:07 2024 +0100
@@ -226,6 +226,7 @@
     
     if(nsapi_rq && !nsapi_rq->finished) {
         // request processing still ongoing and SAFs will be executed
+        // TODO: future ucx cxMapRemove returns, if it actually removed something, therefore use just cxMapRemove
         if(!cxMapRemoveAndGet(nsapi_rq->resources, cx_hash_key_str(respool->name))) {
             log_ereport(LOG_FAILURE, "resourcepool_free: cannot remove resource from request: potential double free");
         }

mercurial