kbss-cvut / s-pipes

Tool for execution of RDF-based pipelines.
GNU Lesser General Public License v3.0
4 stars 5 forks source link

Use annotations to inject LOG #244

Closed blcham closed 1 month ago

blcham commented 2 months ago

Refactor code to use annotation instead of definition in a private static field.

palagdan commented 1 month ago

@blcham I suggest to close this issue because of impossibility to inject logger with custom field name like LOG(in our case). It has reserved name 'log' . From: https://projectlombok.org/features/log documentation.

You put the variant of @Log on your class (whichever one applies to the logging system you use); you then have a static final log field, initialized as is the commonly prescribed way for the logging framework you use, which you can then use to write log statements.

blcham commented 1 month ago

I do not care about renaming LOG --> log if this is the issue.

palagdan commented 1 month ago

@blcham ok then, I will also refactor LOG variable to log in each class where it is used.

blcham commented 1 month ago

Changes were reverted.