Re: DailyRollingFileAppender - why isn't rollOver protected
Thanks. I already told Ceki my opinion about log4j (I think it is really great and I love it). I would gladly check the file in jakarta-log4j and tell you if it solves my problem or any other feedback I have. However, I have to admit that I don't know how to find it. I never browsed the CVS before, the best I could find is: http://cvs.apache.org/viewcvs.cgi/jakarta-log4j/src/java/org/apache/log4j/Da ilyRollingFileAppender.java but this claimed to be version 1.24 and suffer from all the problems I have in version 1.28. can you help me finding the right file. - Original Message - From: "Paul Smith" <[EMAIL PROTECTED]> To: "'Log4J Users List'" <[EMAIL PROTECTED]> Sent: Sunday, June 29, 2003 11:27 PM Subject: RE: DailyRollingFileAppender - why isn't rollOver protected > Hi Avner, > > You'll be pleased to know that Ceki has done a LOT of work on refactoring > the Rolling appender stuff for Log4j 1.3 (not yet released). I can't give > 100% guarantee that the new changes are what you are looking for, but > suffice to say his new design should make it a lot easier to customize > behaviour. > > Perhaps you could check out the jakarta-log4j module and see his new changes > too see if it meets your needs? (we always love feedback! :) ) > > cheers, > > Paul Smith > > > -Original Message- > > From: Avner BenHanoch [mailto:[EMAIL PROTECTED] > > Sent: Sunday, 29 June 2003 9:38 PM > > To: Log4J Users List > > Subject: DailyRollingFileAppender - why isn't rollOver protected > > > > > > I need to customize DailyRollingFileAppender, so rollOver > > will check if > > renaming the file is safe, before it renames it (because my > > application > > enable the user to view/query the log files). > > What I need is that in case the file is being read, don't > > rollOver, instead > > try again on the next time a record is appended. > > Unfortunately, this is not > > possible, because rollOver is not protected. To my opinion > > it should be > > protected, otherwise there is no real option to extend the class. > > > > If one want more details, then here they come: At first I > > wanted to do that > > by inheriting DailyRollingFileAppender and overriding > > subAppend. Then I > > noticed that it won't work (because subAppend schedules the > > time for next > > rollOver, even if current rollOver failed - so it won't > > happend on next time > > a record is appended). Then I wanted to override rollOver to > > raise a flag > > that will be inspected on next subAppend, to see if a > > rollOver is needed. > > Unfortunately, this is not possible, because rollOver is not > > protected. > > What I did to solve my problem is a little bit ugly. I copied > > DailyRollingFileAppender to my package without changes, changed its > > packagename accordingly, then I could extend it from a file > > in my package > > and called rollOver. this worked fine, but I think it is > > better if one > > could change rollOver or subAppend in DRFA, so it will be > > easier to extend > > the file. > > > > thanks, > > Avner > > > > > > > > - > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > - > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: DailyRollingFileAppender - why isn't rollOver protected
Hi Avner, You'll be pleased to know that Ceki has done a LOT of work on refactoring the Rolling appender stuff for Log4j 1.3 (not yet released). I can't give 100% guarantee that the new changes are what you are looking for, but suffice to say his new design should make it a lot easier to customize behaviour. Perhaps you could check out the jakarta-log4j module and see his new changes too see if it meets your needs? (we always love feedback! :) ) cheers, Paul Smith > -Original Message- > From: Avner BenHanoch [mailto:[EMAIL PROTECTED] > Sent: Sunday, 29 June 2003 9:38 PM > To: Log4J Users List > Subject: DailyRollingFileAppender - why isn't rollOver protected > > > I need to customize DailyRollingFileAppender, so rollOver > will check if > renaming the file is safe, before it renames it (because my > application > enable the user to view/query the log files). > What I need is that in case the file is being read, don't > rollOver, instead > try again on the next time a record is appended. > Unfortunately, this is not > possible, because rollOver is not protected. To my opinion > it should be > protected, otherwise there is no real option to extend the class. > > If one want more details, then here they come: At first I > wanted to do that > by inheriting DailyRollingFileAppender and overriding > subAppend. Then I > noticed that it won't work (because subAppend schedules the > time for next > rollOver, even if current rollOver failed - so it won't > happend on next time > a record is appended). Then I wanted to override rollOver to > raise a flag > that will be inspected on next subAppend, to see if a > rollOver is needed. > Unfortunately, this is not possible, because rollOver is not > protected. > What I did to solve my problem is a little bit ugly. I copied > DailyRollingFileAppender to my package without changes, changed its > packagename accordingly, then I could extend it from a file > in my package > and called rollOver. this worked fine, but I think it is > better if one > could change rollOver or subAppend in DRFA, so it will be > easier to extend > the file. > > thanks, > Avner > > > > - > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DailyRollingFileAppender - why isn't rollOver protected
I need to customize DailyRollingFileAppender, so rollOver will check if renaming the file is safe, before it renames it (because my application enable the user to view/query the log files). What I need is that in case the file is being read, don't rollOver, instead try again on the next time a record is appended. Unfortunately, this is not possible, because rollOver is not protected. To my opinion it should be protected, otherwise there is no real option to extend the class. If one want more details, then here they come: At first I wanted to do that by inheriting DailyRollingFileAppender and overriding subAppend. Then I noticed that it won't work (because subAppend schedules the time for next rollOver, even if current rollOver failed - so it won't happend on next time a record is appended). Then I wanted to override rollOver to raise a flag that will be inspected on next subAppend, to see if a rollOver is needed. Unfortunately, this is not possible, because rollOver is not protected. What I did to solve my problem is a little bit ugly. I copied DailyRollingFileAppender to my package without changes, changed its packagename accordingly, then I could extend it from a file in my package and called rollOver. this worked fine, but I think it is better if one could change rollOver or subAppend in DRFA, so it will be easier to extend the file. thanks, Avner - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]