switch to a new logging framework (Log4Net) (1 Viewer)

HomeY

Test Group
  • Team MediaPortal
  • February 23, 2008
    6,475
    4,645
    49
    ::1
    Home Country
    Netherlands Netherlands
    Hi,
    Been testing this for some hours & seems good after some start & stop(8) i have 5.log* & one -log which is fine

    Regards
    Roy

    Same results here. Seems to work fine.
     

    gibman

    Retired Team Member
  • Premium Supporter
  • October 4, 2006
    2,998
    1,372
    Aarhus
    Home Country
    Denmark Denmark
    about log4net.. we really need this in tve3.5 as well as in tve3.5@mp2 :)


    <morpheus_xx> who can help for switching logging to Log4Net ?
    <tourettes> there is a branch for MP1 / TVE3 - so it would be probably wise to get that thing to sorted out in one place and then merge to TVE35 instead of doing thw work twice


    about logging performance, how does it work behinds the curtains ?
    ideally it should be an "in memory queue" of log items.
    then a threaded static logger class would pop those queued log items and commit them to disk, one at a time.
    this would make calls to log very cheap performance wise.

    so an encapsulation of log4net would be needed here.

    I've already done similar work for an asynch mailer class at work.
    send mail to queue and process it in a thread, then dispatch mails one at a time.
    could easily be modded to take care of logging instead of mailing.

    /gibman
     
    Last edited:

    tourettes

    Retired Team Member
  • Premium Supporter
  • January 7, 2005
    17,301
    4,800
    also when will we get log options in setuptv?
    verbosity, rotation, max # logs to keep, .... :)
    It is all implemented in this branch :).
    One logger for all program's and compatible with the "old" implementation (=> no recompiling of the plugin's needed).

    iloop, someone still needs to implement the log levels for every single logging call on TVE side for the verbose to work.
     

    gibman

    Retired Team Member
  • Premium Supporter
  • October 4, 2006
    2,998
    1,372
    Aarhus
    Home Country
    Denmark Denmark
    iloop, someone still needs to implement the log levels for every single logging call on TVE side for the verbose to work.
    I can do this.
    @gibman:
    Is it OK for you to merge this branch with your TVE3.5 branch?
    If yes, who will do it?

    yeah feel free to do it :)

    just remember there are 2 tve3.5 branches .. confusing right ?
    @Sebastiii knows more about this :)

    Once this is done I could goldplate it with the above mentioned threading model for performace goals.

    /gibman
     

    tourettes

    Retired Team Member
  • Premium Supporter
  • January 7, 2005
    17,301
    4,800
    Once this is done I could goldplate it with the above mentioned threading model for performace goals

    If log4net is any sane logging framework it already should have the actual loggin in a separate thread and only the message queuing is done in the calling thread. Bavarian probably knows if this is the case.
     

    infinite.loop

    Retired Team Member
  • Premium Supporter
  • December 26, 2004
    16,163
    4,133
    127.0.0.1
    Home Country
    Austria Austria
    It is all implemented in this branch :).
    One logger for all program's and compatible with the "old" implementation (=> no recompiling of the plugin's needed).
    So in that branch all the config options are availabe in the setuptv gui?
    in 1.3.0 the configfile has to be manually edited, right?
     

    Users who are viewing this thread

    Top Bottom