waku-org/nwaku
1
0
Fork 0
mirror of https://github.com/waku-org/nwaku.git synced 2025-01-27 07:06:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

postgres_driver: remove duplicated const MaxNumConns. (#2106)

Browse Source
This commit is contained in:
Ivan FB 2023-10-04 15:28:07 +02:00 committed by GitHub
parent f05528d4be
commit c74abe131d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
waku/waku_archive/driver/builder.nim
View File

@ -81,8 +81,7 @@ proc new*(T: type ArchiveDriver,
of "postgres": of "postgres":
when defined(postgres): when defined(postgres):
const MaxNumConns = 5 #TODO: we may need to set that from app args (maybe?) let res = PostgresDriver.new(dbUrl = url, onErrAction = onErrAction)
let res = PostgresDriver.new(url, MaxNumConns, onErrAction)
if res.isErr(): if res.isErr():
return err("failed to init postgres archive driver: " & res.error) return err("failed to init postgres archive driver: " & res.error)

4
waku/waku_archive/driver/postgres_driver/postgres_driver.nim
View File

@ -40,11 +40,11 @@ proc insertRow(): string =
"""INSERT INTO messages (id, storedAt, contentTopic, payload, pubsubTopic, """INSERT INTO messages (id, storedAt, contentTopic, payload, pubsubTopic,
version, timestamp) VALUES ($1, $2, $3, $4, $5, $6, $7);""" version, timestamp) VALUES ($1, $2, $3, $4, $5, $6, $7);"""
const DefaultMaxConnections = 5 const MaxNumConns = 5 #TODO: we may need to set that from app args (maybe?)
proc new*(T: type PostgresDriver, proc new*(T: type PostgresDriver,
dbUrl: string, dbUrl: string,
maxConnections: int = DefaultMaxConnections, maxConnections: int = MaxNumConns,
onErrAction: OnErrHandler = nil): onErrAction: OnErrHandler = nil):
ArchiveDriverResult[T] = ArchiveDriverResult[T] =