Thank you Maxim!

I'd gladly test this.
Is there an standard/easy (maven?) way to test this in my application?

-Rob

On Wed, Oct 24, 2018 at 9:52 AM Maxim Solodovnik <solomax...@gmail.com>
wrote:

> Here is the JIRA [1]
>
> I was able to reproduce this issue running one of our tests in a bash loop
> Original 8.2.0-SNAPSHOT usually hang on <10th iteration
>
> After this [1] fix, it flawlessly works 201 iterations
> I would appreciate if you can review/test these changes
>
> [1] https://issues.apache.org/jira/browse/WICKET-6603
> [2] https://github.com/apache/wicket/pull/300
>
>
> On Fri, 12 Oct 2018 at 21:59, Rob Audenaerde <rob.audenae...@gmail.com>
> wrote:
>
> > BTW: I currently worked around it by (very ugly) adding a
> > Thread.sleep(1000) between the last test-action in Wicket and the
> > Jetty-shutdown. So it seems a timing issue (which probably explains why
> > Maxim has been seeing Jenkins failing occasionally).
> >
> > Thanks for checking into this!
> >
> > -Rob
> >
> > On Fri, Oct 12, 2018 at 4:08 PM Rob Audenaerde <rob.audenae...@gmail.com
> >
> > wrote:
> >
> > > Is there a simple way to disable the AsynchronousDataStore (for now)?
> > >
> > > On Fri, Oct 12, 2018 at 3:31 PM Maxim Solodovnik <solomax...@gmail.com
> >
> > > wrote:
> > >
> > >> I'm occasionally able to see this error on Apache Jenkins, but
> > >> unfortunately there is no clear steps to reproduce
> > >> I was unable to find what is wrong, will spend some more time on this
> > >> next week
> > >> On Fri, 12 Oct 2018 at 17:20, Martin Grigorov <mgrigo...@apache.org>
> > >> wrote:
> > >> >
> > >> > Hi,
> > >> >
> > >> > When the application stops Application#destroy() should call
> > >> > AsynchronousDataStore#destroy() (
> > >> >
> > >>
> >
> https://github.com/apache/wicket/blob/6ee5313d405fbbdd4ce9d9a03ccb526358834fca/wicket-core/src/main/java/org/apache/wicket/pageStore/AsynchronousDataStore.java#L101
> > >> > )
> > >> > There it will interrupt the page saving thread and it should stop:
> > >> >
> > >>
> >
> https://github.com/apache/wicket/blob/6ee5313d405fbbdd4ce9d9a03ccb526358834fca/wicket-core/src/main/java/org/apache/wicket/pageStore/AsynchronousDataStore.java#L325
> > >> >
> > >> > I do not remember changes in this area of the code in 8.x but it is
> > not
> > >> > impossible.
> > >> >
> > >> > On Fri, Oct 12, 2018 at 12:32 PM Rob Audenaerde <
> > >> rob.audenae...@gmail.com>
> > >> > wrote:
> > >> >
> > >> > > Hi all,
> > >> > >
> > >> > > I'm experiencing some jetty-shutdown problems after upgrading from
> > >> 7.9 to
> > >> > > 8.1. See the stacktrace below.
> > >> > >
> > >> > > Is there a simple remedy for this? (or a known cause?) It is
> > annoying
> > >> > > because this error prevents Jetty from shutting down, and I use
> this
> > >> in my
> > >> > > tests, so my tests never end... :(
> > >> > >
> > >> > > Btw. I'm on java 10.0.2 and jetty version 9.2.5.v20141112
> > >> > >
> > >> > >
> > >> > > 2018-10-12 11:24:25,738 ERROR - AsynchronousDataStore      -
> > >> > > java.lang.InterruptedException
> > >> > >         at
> > >> > >
> > >> > >
> > >>
> >
> java.base/java.util.concurrent.locks.AbstractQueuedSynchronizer.acquireInterruptibly(AbstractQueuedSynchronizer.java:1259)
> > >> > >         at
> > >> > >
> > >> > >
> > >>
> >
> java.base/java.util.concurrent.locks.ReentrantLock.lockInterruptibly(ReentrantLock.java:317)
> > >> > >         at
> > >> > >
> > >> > >
> > >>
> >
> java.base/java.util.concurrent.LinkedBlockingQueue.offer(LinkedBlockingQueue.java:373)
> > >> > >         at
> > >> > >
> > >> > >
> > >>
> >
> org.apache.wicket.pageStore.AsynchronousDataStore.storeData(AsynchronousDataStore.java:207)
> > >> > >         at
> > >> > >
> > >> > >
> > >>
> >
> org.apache.wicket.pageStore.AbstractPageStore.storePageData(AbstractPageStore.java:119)
> > >> > >         at
> > >> > >
> > >> > >
> > >>
> >
> org.apache.wicket.pageStore.DefaultPageStore.storePage(DefaultPageStore.java:66)
> > >> > >         at
> > >> > >
> > >> > >
> > >>
> >
> org.apache.wicket.pageStore.AsynchronousPageStore$PageSavingRunnable.run(AsynchronousPageStore.java:225)
> > >> > >         at java.base/java.lang.Thread.run(Thread.java:844)
> > >> > >
> > >>
> > >>
> > >>
> > >> --
> > >> WBR
> > >> Maxim aka solomax
> > >>
> > >> ---------------------------------------------------------------------
> > >> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
> > >> For additional commands, e-mail: users-h...@wicket.apache.org
> > >>
> > >>
> >
>
>
> --
> WBR
> Maxim aka solomax
>

Reply via email to