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

Get rid of FileSystemCacheThreshold parameter [CORE6332] #6573

Closed
firebird-issue-importer opened this issue Jun 14, 2020 · 3 comments
Closed

Get rid of FileSystemCacheThreshold parameter [CORE6332] #6573

firebird-issue-importer opened this issue Jun 14, 2020 · 3 comments

Comments

@firebird-issue-importer

Submitted by: @WarmBooter

Votes: 1

Since FileSystemCacheThreshold looks like an attempt to allow a kind of "per database" configuration for filesystem cache usage when there was no databases.conf, I suggest to get rid of it and create a more obvious per database parameter, ie: AllowFilesystemCache (boolean), in Firebird 4.

Commits: 7e61b9f

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 14, 2020

Modified by: @hvlad

assignee: Vlad Khorsun [ hvlad ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Nov 13, 2020

Modified by: @hvlad

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

resolution: Fixed [ 1 ]

Fix Version: 4.0 RC 1 [ 10930 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 20, 2021

Commented by: @hvlad

New boolean per-database setting "UseFileSystemCache" was introduced. It overrides legacy "FileSystemCacheThreshold" setting, that will be removed in the next major Firebird release.

If "UseFileSystemCache" is set, then value of "FileSystemCacheThreshold" is ignored.
If "UseFileSystemCache" is not set, and "FileSystemCacheThreshold" is set then value of "FileSystemCacheThreshold" is in use and accounted by the engine.

See also description at firebird.conf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants