bchapuis commented on issue #775:
URL:
https://github.com/apache/incubator-baremaps/issues/775#issuecomment-1711934064
That's a nice setup ;)
Regarding this issue I think we have to options. One would consist in adding
a command line parameter to the command line (e.g. `--maximum-pool-size 64`).
The other one would consist in adding this parameter to the tileset.json file.
```
{
...
'database': 'jdbc://...',
'maximum-pool-size': 64
}
```
As the need for customization increases, we could to introduce a new
'datasource' field with more configuration parameters and deprecate the
'database' field.
```
{
'datasource': {
'jdbcUrl': 'jdbc://...',
'maximumPoolSize': 64,
'minimumPoolSize': 16,
...
}
}
```
It would be nice to have your feedback on these alternative before doing the
implementation. Personally, I tend to prefer the third option.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
