# Migrate Qserv code to stream-based logging

XMLWordPrintable

## Details

• Type: Story
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
None
• Story Points:
10
• Epic Link:
• Sprint:
DB_W16_01
• Team:
Data Access and Database

## Description

Migrate Qserv code from LOGF_* to LOGS_*.

While doing it, we will also revisit logging levels: in particular we are abusing INFO, most of what is now in INFO should be on DEBUG, in some places where INFO is used to cover unusual conditions, it should go to WARNING.

Further, we will unify how we initialize logging structures. Per discussions at 2015/12/09 Qserv meeting, we like best

 LOG_LOGGER _log = LOG_GET("lsst.qserv..")

in anonymous namespace in cc files. Logging from .hh files is strongly discouraged.

This involves changing ~600 places.

## Activity

Hide
Jacek Becla added a comment -

Andy, can you review. It is pretty large:

 91 files changed, 1052 insertions(+), 1001 deletions(-)

, but the changes are not overly complicated . Thanks.

Show
Jacek Becla added a comment - Andy, can you review. It is pretty large: 91 files changed, 1052 insertions(+), 1001 deletions(-) , but the changes are not overly complicated . Thanks.
Hide
Andy Salnikov added a comment -

I'm done reviewing, few comments left in PR. There are very many changes so there is a chance I missed something. Good thing is that if it compiles then it should not crash any more.
I think John is ready to merge his ticket which has few new logging messages, you will need to re-check it after rebasing.

Show
Andy Salnikov added a comment - I'm done reviewing, few comments left in PR. There are very many changes so there is a chance I missed something. Good thing is that if it compiles then it should not crash any more. I think John is ready to merge his ticket which has few new logging messages, you will need to re-check it after rebasing.

## People

• Assignee:
Fritz Mueller
Reporter:
Fritz Mueller
Reviewers:
Andy Salnikov
Watchers:
Andy Salnikov, Jacek Becla
• Votes:
0 Vote for this issue
Watchers:
2 Start watching this issue

## Dates

• Created:
Updated:
Resolved: