search

EVGENIAST / cumulusrdf

Automatically exported from code.google.com/p/cumulusrdf
0 stars 0 forks source link

Logging framework + message catalog #25

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Two enhancements are included in this issue:

1) Refactor code in order to use a more flexible and fast logging framework 
(log4j or logback). At the moment JULI is used which is optimized in Tomcat but 
relies on standard java util logging which offers a limited set of 
capabilities. 

2) A message catalog, that would consiste in an enumerative interface 
(IMessageCatalog) where all CumulusRDF messages are defined. That would allow a 
structured log with (for example) messages like this

...
2014-01-15 17:05:42,105 INFO  <CRDF-00011> : CUMULUS-RDF 1.0.0 open for 
e-business.  
...

As you can see other than having all messages classified, we could associate a 
code with each message and, for relevant messages (e.g. errors), we could 
create a Wiki page with something like:

- Code: CDRF-000034
- Level: ERROR
- Message: Malformed configuration file.
- Suggested action: check your configuration file blabalbla

I know, that would require a more effort each time we need to write an 
additional log message, but at the same time it will provide a very powerful 
and meaningful log subsystem

Original issue reported on code.google.com by a.gazzarini@gmail.com on 27 Jan 2014 at 10:11

GoogleCodeExporter commented 8 years ago 
Ok, I'm changing this issue to started. 
I'll create a first implmentation with few messages and on top of that we will 
adjust things dependings on our needs.

Original comment by a.gazzarini@gmail.com on 29 Jan 2014 at 4:46

GoogleCodeExporter commented 8 years ago 
a MessageCatalog + a Log (wrapper) class over SLF4j have been created  

Usage (IMHO) is very simple...see 

edu.kit.aifb.cumulus.webapp.Listener

NB: not yet committed. When I will do I'll close the issue.

Original comment by a.gazzarini@gmail.com on 31 Jan 2014 at 11:27

GoogleCodeExporter commented 8 years ago 
This issue was closed by revision r804.

Original comment by a.gazzarini@gmail.com on 31 Jan 2014 at 2:00

GoogleCodeExporter commented 8 years ago 
Issue fixed. See comment 
https://code.google.com/p/cumulusrdf/issues/detail?id=25#c2 for a description 
of changes.

Original comment by a.gazzarini@gmail.com on 31 Jan 2014 at 2:01