Re: [log4j] Java 8 and java.time

2018-11-04 Thread Remko Popma
> On Nov 5, 2018, at 2:33, Gary Gregory wrote: > > Some notes. > > Java 8 provides a java.time package. It would be nice to reuse it instead > of keeping our own Java 7-based code. > > We provide a MutableInstant class akin to the final immutable thread-safe > java.time.Instant class. > > Ou

Re: [log4j] Java 8 and java.time

2018-11-04 Thread Ralph Goers
I am sure we can find a way to eliminate our custom time code or base it on java.time. Whether it performs better or not would require that it be tested and compared. It wouldn’t be a simple matter to create a JMH benchmark and create our time formatting against java.time. Ralph > On Nov 4, 20

[log4j] Java 8 and java.time

2018-11-04 Thread Gary Gregory
Some notes. Java 8 provides a java.time package. It would be nice to reuse it instead of keeping our own Java 7-based code. We provide a MutableInstant class akin to the final immutable thread-safe java.time.Instant class. Our class is not interchangeable with java.time.Instant but it could be a

Re: [log4j] Help needed with master TimeFilterTest

2018-11-04 Thread Gary Gregory
On Sun, Nov 4, 2018 at 9:53 AM Ralph Goers wrote: > Did you make a Jira for that? I would like to focus on 3.0 but I have so > little time… > > FWIW, I just ran the build on my laptop and had no problems. > I did not make a JIRA. The Jenkins build fails as well. java.lang.AssertionError: expect

Re: [log4j] Help needed with master TimeFilterTest

2018-11-04 Thread Ralph Goers
Did you make a Jira for that? I would like to focus on 3.0 but I have so little time… FWIW, I just ran the build on my laptop and had no problems. Ralph > On Nov 4, 2018, at 9:37 AM, Gary Gregory wrote: > > On Sun, Nov 4, 2018 at 9:34 AM Matt Sicker wrote: > >> The day of daylight saving ti

Re: [log4j] Help needed with master TimeFilterTest

2018-11-04 Thread Gary Gregory
On Sun, Nov 4, 2018 at 9:34 AM Matt Sicker wrote: > The day of daylight saving time ending or starting has historically caused > failing unit tests. I’m not sure if that was ever fixed. Time is hard 😓 > I am hoping that once we put more effort into 3.0 we can move to using the Java 8 time APIs.

Re: [log4j] Help needed with master TimeFilterTest

2018-11-04 Thread Matt Sicker
The day of daylight saving time ending or starting has historically caused failing unit tests. I’m not sure if that was ever fixed. Time is hard 😓 On Sun, Nov 4, 2018 at 10:22, Gary Gregory wrote: > Hi All: > > After commit 65fa087093a6de16e284e3189fda303f4b0111ab in master, the > test TimeFilte

[log4j] Help needed with master TimeFilterTest

2018-11-04 Thread Gary Gregory
Hi All: After commit 65fa087093a6de16e284e3189fda303f4b0111ab in master, the test TimeFilterTest started failing. That commit removed deprecated tests and updated one API call from deprecated to not deprecated. This makes me wonder how the change could have caused this test to fail. Any ideas? W