You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@celix.apache.org by GitBox <gi...@apache.org> on 2020/09/10 10:11:48 UTC
[GitHub] [celix] pnoltes commented on pull request #282: Hotfix/rsa deadlock

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