Menu

#338 SQLeo Start up GUI does nothing

2016.02
closed
Anu
Bug (150)
1
2016-02-13
2016-01-15
PAscal
No

Discussion

  • PAscal

    PAscal - 2016-01-20
    • priority: --> 1
    • Milestone: not planned --> 2016.02
     
  • PAscal

    PAscal - 2016-01-20

    Hello dev team,

    How could we find what is happening here ?

    Note: maybe we can use external tools for tracing, I'm also ready to put in code some specific information messages to understand what is happening (using a flag for enable/disable)

    Thanks

     

    Last edit: PAscal 2016-01-20
  • Anu

    Anu - 2016-01-21

    My analysis from thread dump of the user,
    Deadlock on 2 threads waiting on each other, Main thread is trying to show all components and paint them by locking all components tree of the window and another thread is trying to add new version link on top right meaning its also trying to lock the same components tree or some inner component of the same window which was already locked by previous thread.

    Lets confirm from user if disabling the "check for update" option fixes this problem. As disabling it will not launch any other thread to display the version link, hence there is no other thread blocking the main one,

    After confirmation from the user i will make a fix to add the version link differently to make sure that we add this version link only after the previous window painting/loading is finished so that we dont have threads waiting on each other. Seem like revalidate(),repaint() of version link is blocking the main thread, fix is made but it needs to be tested by user as we cant reproduce.

    @pascal - can you send that user the new jar to test my fix ?

     

    Last edit: Anu 2016-01-21
    • PAscal

      PAscal - 2016-01-23

      I have committed some additionnal println messages to try to understand ...
      Would you like to add some more before I propose it for users testing ?

       

      Last edit: PAscal 2016-01-23
  • Anu

    Anu - 2016-01-21
    • status: open --> accepted
    • assigned_to: Anu
     
  • Anu

    Anu - 2016-01-24

    I tried to display main window in another thread , lets try with this fix if problem is resolved

     
  • PAscal

    PAscal - 2016-01-24
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1 +1 @@
    -see [discution](https://sourceforge.net/p/sqleo/discussion/general/thread/ce7230b9/)
    +see [discussion](https://sourceforge.net/p/sqleo/discussion/general/thread/ce7230b9/)
    
     
  • PAscal

    PAscal - 2016-01-26
    • status: accepted --> tested
     
  • PAscal

    PAscal - 2016-02-13
    • status: tested --> closed
     

Log in to post a comment.