-
-
Notifications
You must be signed in to change notification settings - Fork 487
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update instructions for controlling spatial index transactions
- Loading branch information
1 parent
f52a2fe
commit 6692428
Showing
1 changed file
with
18 additions
and
13 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6692428
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.
Should we set the default value to 1 as it will be better for DBCP (used probably most of the time) and will avoid the DB connection issue ?
6692428
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.
Yes - seems little point to leaving it at 1000 as that would possibly lead sites into trouble with DB connections. I was a bit hesitant to change it in 2.8 until spoken to you and other devs as must have time to test in 2.8 RC process - should be ok with RC1 and RC2 perhaps still to come?
6692428
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.
Do you know a sequence of actions to trigger the error ? (stop/start the database when geonetwork is running with editing is not enough). I've changed the settings on the node I'm having the issue. I will keep you up to date if the problem occur again.