celix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [celix] pnoltes opened a new pull request #209: Feature/refactored logging service
Date Tue, 28 Apr 2020 19:58:16 GMT

pnoltes opened a new pull request #209:
URL: https://github.com/apache/celix/pull/209

   Refactored the logging facilities. This includes a new log service concept and logging
of the framework. 
   Logging in Celix was IMO not very useable. The refactored implementation is inspired by
the OSGi Log Service specification, but does not follow it 100%. Mainly because I think A
LogFactory does not fit in a non memory managed environment. 
   Instead I decided to use a service on demand concept. So that bundles can request their
own log services and these instances can be used the differentiate between loggers. 
   Also the backup logging facilities (log to stdout/stderr) foor framework logging, logging
with only a log_helper and logging with a log_admin but no available log sinks is aligned
using a  set of celix_logUtils functions. This in many different situatie the logging looks
the same.
   Additionally changes: 
   - Updates the default behaviour for closing service trackers. This was done on a separately
spawned thread, but because service listeners events are triggered outside of locks that is
not needed anymore.
   - Fixes an issues that listener hooks where not retroactively triggered when they where
added to the service registry.
   - Removes the framework cpputest + mocks. Refactoring the framework logging was near impossible
with the existing mocks and because we want to move to google test I considered them deprecated.
Note that we have already quite some google tests, but need to do more to increase coverage.

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:

View raw message