Dirk's improvements#8
Conversation
anjohnson
left a comment
There was a problem hiding this comment.
These improvements generally seem worthwhile but are currently incomplete. The changes need documenting, especially to the log format, and multi-client support for the JSON output format code is missing — I'm not even sure if that will still build or run properly.
Conflicts: caPutLogApp/caPutLogTask.c
Conflicts: caPutLogApp/caPutLogTask.c
9c05af7 to
f65035a
Compare
|
I have force-pushed an update with several of the issues fixed. In particular changes are documented now. |
|
I could implement the support for multiple servers for the JSON version too. But there is still a problem in caPutLogAs.c (global variables!) which prevents using both, the old and the JSON method at the same time. Instead of fixing that, I consider integrating the old implementation into the new one so that it can support JSON and the legacy format. |
|
Ok, if that is the intention. I noticed this when implementing the "multiple log server" feature and thought that there may be one log server that understands JSON and another one that understands the old format. |
b730cff to
140182e
Compare
140182e to
d14203a
Compare
Some cumulative improvements made at PSI over the last years:
New features:
caPutLogReconffunctional ) and show current log mode incaPutLogShow.Changes in message formatting: