Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memory leak when use SHOW GRANTS on new empty database (3.0.1 & 4.0; SS & SC) [CORE5294] #5572

Closed
firebird-issue-importer opened this issue Jun 28, 2016 · 7 comments

Comments

@firebird-issue-importer

Submitted by: @pavel-zotov

Attachments:
memory-leak-when-use-show-grants-in-FB_301-or-FB_400.7z
firebird.conf.files.7z

1) Run ProcessExplorer or similar tool for monitoring process resources

2) create empty database, add it to databases.conf (aliases.conf for 2.5)

3) Run following .sql script several times:

set wng off;
set list on;
set echo on;

connect 'localhost:e25' user sysdba password 'masterke';
show grants;
quit;

Sample of batch for this ('fbc' variable stored path to different FB binaries on the same host ):

@echo off
set fbc=C:\MIX\firebird\fb25sS\bin
::set fbc=C:\MIX\firebird\fb25\bin
::set fbc=C:\MIX\firebird\fb30
::set fbc=C:\MIX\firebird\fb30sC
::set fbc=C:\MIX\firebird\fb40
::set fbc=C:\MIX\firebird\fb40sc

for /l %%i in (1,1,10) do (
echo %%i
%fbc%\isql -q -i memotrouble.sql 1>memotrouble.log 2>&1
)

On WI-T4.0.0.267 and WI-V3.0.1.32542 you will find noticable memory leak for FB process (I've tested bothSS and SC).
No such trouble on WI-V2.5.6.27017 SC or SS.

Please see screenshots in attached .7z file.

Commits: a8fdbdf ee105dd

====== Test Details ======

No way to detect memory leak using FB monitoring; it seems that Python 'psutil' package could help but it's not supplied in standard Python kit.

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jun 28, 2016

Modified by: @pavel-zotov

Attachment: memory-leak-when-use-show-grants-in-FB_301-or-FB_400.7z [ 12980 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jun 28, 2016

Modified by: @pavel-zotov

Attachment: firebird.conf.files.7z [ 12981 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jun 28, 2016

Commented by: @pavel-zotov

PS. All used firebird.conf files see in attached "firebird.conf.files.7z"

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jun 30, 2016

Commented by: @AlexPeshkoff

Confirmed on linux with default configs

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jun 30, 2016

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jul 6, 2016

Modified by: @AlexPeshkoff

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 3.0.1 [ 10730 ]

Fix Version: 4.0 Alpha 1 [ 10731 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Aug 1, 2016

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Cannot be tested

Test Details: No way to detect memory leak using FB monitoring; it seems that Python 'psutil' package could help but it's not supplied in standard Python kit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment