celix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [celix] pnoltes commented on pull request #282: Hotfix/rsa deadlock
Date Thu, 10 Sep 2020 10:11:48 GMT

pnoltes commented on pull request #282:
URL: https://github.com/apache/celix/pull/282#issuecomment-690132803


   > The tests logs the following:
   > 
   > ```
   > [2020-09-10T11:15:03] [warning] [celix_framework] [serviceRegistry_logWarningServiceReferenceUsageCount:601]
Service Reference destroyed with usage count is 2, expected 0. Look for missing bundleContext_ungetService
calls.
   > [2020-09-10T11:15:03] [warning] [celix_framework] [serviceRegistry_logWarningServiceReferenceUsageCount:620]
Previous Dangling service reference warnings caused by bundle 'apache_celix_rs_topology_manager',
for service 'unknown', provided by bundle 'unknown'
   > [2020-09-10T11:15:03] [warning] [celix_framework] [serviceRegistry_logWarningServiceReferenceUsageCount:601]
Service Reference destroyed with usage count is 2, expected 0. Look for missing bundleContext_ungetService
calls.
   > [2020-09-10T11:15:03] [warning] [celix_framework] [serviceRegistry_logWarningServiceReferenceUsageCount:620]
Previous Dangling service reference warnings caused by bundle 'apache_celix_rs_topology_manager',
for service 'unknown', provided by bundle 'unknown'
   > [2020-09-10T11:15:03] [warning] [celix_framework] [serviceRegistry_logWarningServiceReferenceUsageCount:601]
Service Reference destroyed with usage count is 2, expected 0. Look for missing bundleContext_ungetService
calls.
   > [2020-09-10T11:15:03] [warning] [celix_framework] [serviceRegistry_logWarningServiceReferenceUsageCount:620]
Previous Dangling service reference warnings caused by bundle 'apache_celix_rsa_discovery_configured',
for service 'unknown', provided by bundle 'unknown'
   > ```
   > 
   > Are these acceptable in tests?
   
   Good catch. Hmm, I would say no. 
   Seems like an issue in the remote service topology manager. Most of the remote service
code is not using the update service registration / tracking api (with a celix_ prefix). 
   
   This can result in dangling service reference. This should be solved, but IMO not in this
PR.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message