FlameRobin 0.9.3.7 is taking much longer to connect to a database than the previous version 0.9.3.6

mariuz / flamerobin

FlameRobin is a database administration tool for Firebird RDBMS. Our goal is to build a tool that is: lightweight (small footprint, fast execution) cross-platform (Linux, Windows, Mac OS X, FreeBSD) dependent only on other Open Source software

http://flamerobin.org

MIT License

216 stars 66 forks source link

FlameRobin 0.9.3.7 is taking much longer to connect to a database than the previous version 0.9.3.6 #181

Closed luronumen closed 3 years ago

luronumen commented 3 years ago

ACTUAL RESULT

FlameRobin 0.9.3.7 is taking much longer to connect to a database than the previous version 0.9.3.6;
The Not Responding screen remains for several seconds until it is able to connect to the database:

NotResponding

EXPECTED RESULT:

The time to connect to a database should be similar to the previous version

IMPORTANT NOTES:

This issue is best seen on a high latency network.

arvanus commented 3 years ago

same here

luronumen commented 3 years ago

Hi @Jdochoa

Did you get to find the root cause of this issue?

Best Regards, Luciano

Jdochoa commented 3 years ago

Hi @Jdochoa

Did you get to find the root cause of this issue?

Best Regards, Luciano

I work on this.

luronumen commented 3 years ago

Hi @Jdochoa Did you get to find the root cause of this issue? Best Regards, Luciano

I work on this.

Thank you very much @Jdochoa !

arvanus commented 3 years ago

@Jdochoa can you take a look at this?

Jdochoa commented 3 years ago

Hi @Jdochoa

Did you get to find the root cause of this issue?

Best Regards, Luciano

Yes, I have this pending review.

./jo

Jdochoa commented 3 years ago

Hi @arvanus, @luronumen

I fix this, please view my repository

./jo

arvanus commented 3 years ago

Still freeze for remote DB

luronumen commented 3 years ago

Hi @arvanus, @luronumen

I fix this, please view my repository

./jo

Hi @Jdochoa

Could you please generate a compiled version for me? I do not have the environment setup to compile the FlameRobin.

Thanks in advance, Luciano

Jdochoa commented 3 years ago

Hi @arvanus @luronumen Can you try this one?

https://drive.google.com/file/d/1POLlDlhV-l9R3YCBYJ4maywFkd8tajeq/view?usp=sharing

Please.

./jo

luronumen commented 3 years ago

Hi @arvanus @luronumen Can you try this one?

https://drive.google.com/file/d/1POLlDlhV-l9R3YCBYJ4maywFkd8tajeq/view?usp=sharing

Please.

./jo

Hi @Jdochoa

This issue is no longer reproducible on this version but the following regression (see issue #65 ) is happening:

Capture

arvanus commented 3 years ago

@Jdochoa fixed here Sorry, I had messed the git repo and wasn't with the latest commit, it's not hanging now

arvanus commented 3 years ago

Please do a pull request :)

luronumen commented 3 years ago

Retest result on FlameRobin 0.9.3.8 Snapshot: Passed!

Thank you very much for the fix!