Synchronous standbys

[zilder]

What

A wrapper for synchronous_standby_names server option.

Usage

from testgres import get_new_node, First

master = get_new_node().init().start()
with master.replicate().start() as standby:
    master.append_conf("synchronous_commit = remote_apply")
    master.set_synchronous_standbys(First(1, [standby]))
    master.restart()

[codecov-io]

Codecov Report

Merging #46 into master will increase coverage by 0.01%.
The diff coverage is 97.67%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   97.12%   97.13%   +0.01%     
==========================================
  Files          17       18       +1     
  Lines        1529     1572      +43     
==========================================
+ Hits         1485     1527      +42     
- Misses         44       45       +1

Impacted Files	Coverage Δ
testgres/__init__.py	100% <100%> (ø)	⬆️
testgres/standby.py	100% <100%> (ø)
tests/test_simple.py	99.81% <100%> (ø)	⬆️
testgres/node.py	96.77% <90%> (-0.18%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 72d0373...24b0e57. Read the comment docs.

[funbringer]

IMHO those names are too generic to be exported at top level.

[ildus]

I think names are ok but should be used with the of module, like standby.First

[funbringer]

@ildus I agree.

[funbringer]

I think we should replace First and Any with a Enum-based class.

[funbringer]

Now we could write

node.append_conf(synchronous_standby_names=First(1, [standby]))

[ildus]

тут мне кажется лучше добавить ключевое слово first, чтобы было явно

[ildus]

I think names are ok but should be used with the of module, like standby.First