-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Use buffersize to reduce database connection when iterate #2724
Conversation
conf/app.ini
Outdated
@@ -184,6 +184,8 @@ SSL_MODE = disable | |||
PATH = data/gitea.db | |||
; For "sqlite3" only. Query timeout | |||
SQLITE_TIMEOUT = 500 | |||
; For interate buffer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Default setting should be mentioned too IMO.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And it should be For iterate buffer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Morlinest the default is 50, this file is an example file.
@daviian done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMHO it is good practice to comment defaults in config files (even in example files). If you search default
in this file, you can find a lot of comments like ... (default false)
or ... default is ...
. If you look at file you should know what happens if you don't set some variables.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Morlinest done.
Codecov Report
@@ Coverage Diff @@
## master #2724 +/- ##
=======================================
Coverage 26.91% 26.91%
=======================================
Files 87 87
Lines 17286 17286
=======================================
Hits 4652 4652
Misses 11955 11955
Partials 679 679
Continue to review full report at Codecov.
|
conf/app.ini
Outdated
@@ -184,6 +184,8 @@ SSL_MODE = disable | |||
PATH = data/gitea.db | |||
; For "sqlite3" only. Query timeout | |||
SQLITE_TIMEOUT = 500 | |||
; For interate buffer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And it should be For iterate buffer
a06f00e
to
6767a59
Compare
LGTM |
LGTM |
No description provided.