fix: don't throw errors on COPY_BOTH results

This change only allows to execute a `START_REPLICATION` command on a
suitable configured database, i.e. the following commands would work:

    conn = psycopg.connect(DSN, replication="database", autocommit=True)
    conn.execute(
        """
        START_REPLICATION SLOT "my_replication_slot"
        LOGICAL 0/0 (proto_version '1',publication_names 'my_publication')
        """)
    <psycopg.Cursor [COPY_BOTH] [ACTIVE] at 0x...>

instead of gettgin an error such as  "COPY cannot be used with this method".

Please check https://github.com/psycopg/psycopg/issues/71#issuecomment-3393722855
for the description of the db configuration required to make this command work.

No further support for logical replication is planned at this moment, but this
change allows to play with replication without needing to go down at libpq
level to execute replication commands.

Merge pull request #1218 from dvarrazzo/feat/update-loaders

Allow to change loaders in results already returned

feat: allow to change loaders in results already returned

Close #884

Merge pull request #1184 from immortalcodes/feat-store-state-in-transaction

Added transaction attribute to record rollback/commit

docs: improvements to Transaction.status documentation

fix: make Transaction.Status a string enum

docs: add documentation for Transaction status

refactor: remove enter/exit attributes from transaction

feat: add Transaction attribute to record the transaction state

Merge pull request #1177 from adityatoshniwal/master

feat: add support for setting cursor result set position

fix: improve set_result() proposal:

- make it async on async cursors (consistently with `results()`,
  inconsistently with `nextset()`, but the latter being sync on async
  cursors was a design mistake);
- raise IndexError, consistently with sequences, better error message;
- fix test, which were simply broken and the OP didn't bother to fix
  them;
- documentation improved.

feat: add support for setting cursor result set position for multi-result queries

ci: add Python 3.14 tests on macOS, Windows

chore: bump psycopg_pool package version to 3.2.8

chore: bump psycopg package version to 3.2.13

Merge branch 'fix/pool-check-cancel-master'

fix(pool): trap CancelledError more consistently in the pool codebase

Include also places that were left out such as the rollback and the task
scheduling.

Note that we are relaxing the exception handler we had set up to fix the
problem with cancelled clients on wait (#509): we only had to trap
CancelledError additionally but we started managing the whole
BaseException. I don't think that trapping KeyboardException or
SystemExit without re-raising is a good idea (I think that, for
robustness, we should, but then things become very verbose and not
necessarily correct).

fix(pool): manage CancelledError in some exception handling path

If a CancelledError was raised during check the connection would have
been lost. The exception would have bubbled up but likely users are
using some framework swallowing it because nobody reporting the "lost
connections" issue actually reported the CancelledError.

Close #1123
Close #1208

refactor(pool): add _close_connection() method

Improves on the solution for #1124

docs: fix released version in news entry

ci: only run docs workflow on the main repo

https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#example-only-run-job-for-specific-repository

Merge pull request #1207 from deeshantk/fix/improve-dns-error-message

Fix/improve dns error message

docs: mention more informative hostname resolution error message in news

test: test that the host is in the error message on name resolution error

fix(conninfo): improve error message when host resolution fails

Merge pull request #1210

Fix copy concurrency

fix; keep the lock for the entire duration of copy

Avoid errors in concurrent operations on the same connection, from
different cursors.

Close #1210

Fix AsyncCursor.copy concurrency

Apply linter diff

Add test_copy_concurrency and update news

Update docs/news.rst

Co-authored-by: Daniele Varrazzo <daniele.varrazzo@gmail.com>
Update tests/test_copy_async.py

Co-authored-by: Daniele Varrazzo <daniele.varrazzo@gmail.com>
Update tests/test_copy_async.py

Co-authored-by: Daniele Varrazzo <daniele.varrazzo@gmail.com>
Update tests/test_copy_async.py

Co-authored-by: Daniele Varrazzo <daniele.varrazzo@gmail.com>
Simplify execution_log assertion

Revert unnecessary imports

chore(deps): bump the actions group with 2 updates

Bumps the actions group with 2 updates: [pypa/cibuildwheel](https://github.com/pypa/cibuildwheel) and [actions/upload-artifact](https://github.com/actions/upload-artifact).

Updates `pypa/cibuildwheel` from 3.2.0 to 3.2.1
- [Release notes](https://github.com/pypa/cibuildwheel/releases)
- [Changelog](https://github.com/pypa/cibuildwheel/blob/main/docs/changelog.md)
- [Commits](https://github.com/pypa/cibuildwheel/compare/v3.2.0...v3.2.1)

Updates `actions/upload-artifact` from 4 to 5
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v4...v5)

---
updated-dependencies:
- dependency-name: pypa/cibuildwheel
  dependency-version: 3.2.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: actions
- dependency-name: actions/upload-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: actions
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #1202 from dvarrazzo/feat/customize-composite-dumper

Customize composite dumper

docs: add generic object to composite adaptation docs

feat: add make_sequence parameter to register_dumper

Allow to customize the way to extract a sequence from an object to dump
it as composite type.

refactor(composite) mark private classes with leading underscore

fix(c): fix error revealed by Cython 3.2

The error was likely introduced in cec696609b334aa32f871bd342e39b431fc799db
but the Cython compiler didn't flag it as error.

Merge pull request #932 from seandstewart/sean/composite-keword-args

Support loading composite types with keyword arguments

docs: add docs for register_composite's make_instance param

refactor: use make_instance instead of use_keywords

This model is more similar to psycopg2 and allow more freedom to define
how to return a class. There could be a mix of positional and keyword
arguments for example.

feat: support loading composite types with keyword arguments

Changes:

1. Add `KeywordComposite*` classes to support loading composite types with keyword arguments.
2. Some optimizations for the `*BinaryLoader`.
    - Use a generator to unpack (oid, value) pairs.
    - Use a membership check instead of `KeyError` to optimize hot loops when looking up transformers.
3. Generics for the `*InstanceLoader` so mypy can better-track output results.

---

Co-authored by Daniele Varrazzo, mostly rebasing the feature on the
refactoring happening in the composite loading in #1175 and applying
some suggested PR changes.

refactor: cleaner inheritance in record/composite loaders

- Avoid a common base class to reuse the code, use module-level
  functions.
- Make the inheritance lines of text and binary loaders similar.
- Add documentation and notes.

This refactoring is in preparation of the implementation of these
loaders in C, to clarify and simplify things. See #1175.

Merge pull request #1167 from dsuhinin/dmitriy/change-connection-pool-creds

Introduce a way to update connection pool credentials

docs(pool): add API documentation for conninfo/kwargs as callable

feat(pool): allow conninfo/kwargs to be callable

Allow to change connection pool credentials dynamically without
requiring pool re-creation.

Merge pull request #1196 from dvarrazzo/refactor/fast-sequence

Refactor C API to use PyObject_Fast* functions

refactor: look up for the oid for None only if None is actually dumped

perf: use PySequence_Fast function instead of slower alternatives

fix(c): allow set_loader_types to be any sequence

Merge pull request #1199 from dvarrazzo/cython-lint

Add cython code linting

chore: add cython-lint to pre-commit hooks

style: clean up cython code with a pass of cython-lint

perf: use a C var for comparison instead of a Python var

chore: remove unneeded type-ignore comment

Problem solved with Cython 3.1.6 apparently.

Merge pull request #1197 from mliezun/riscv64-support

Add riscv64 support for linux builds

ci: extend CI configuration to include riscv64 architecture support on libpq

ci: update CI configuration to support riscv64 architecture and Rocky Linux

ci: fix proxy-based tests

Move pproxy_fix out of tests package and use it as a script. Importing
it fails in cibuildwheel otherwise

Exclude the proxy test on Win/macOS. They don't work there, but for all
other test they got skipped as side effects of other markers.

test(pool): consider another test flakey

test(pool): consider test flakey

It fails regularly in CI on Windows

docs: add install differences grid

chore: bump psycopg_pool package version to 3.2.7

chore: bump psycopg package version to 3.2.12

Merge pull request #1193 from jerch/fix/no-pinning-copy-text

docs: add note about fixing the copy pinning issue

test(copy): cover the Python implementation in copy pinning tests

fix(copy): restore selecting loader for each item

In the copy refactoring released in 3.2.11 we changed the behaviour to
choosing a dumper at the first non-null value per column and keeping it.

Turns out more people than expected rely on the more flexible behaviour
of passing strings or types and letting postgres do the conversion.

Restore the previous behaviour; however it's possible to use
`copy.set_types()` to set dumpers per column and gain something in
performance.

Fix #1192

ci: enable lint on pull requests

This now matches the tests workflow.

Merge pull request #1067 from psycopg/sqlalchemy-nullpool

Allow integration of psycopg pool with SQLAlchemy

fix(pool): more robust placement of reconnection attempts

The problem reported in #1176 is the close recursion already fixed.
However, because we try to reconnect before closing, we end up scheduling
a reconnection attempt for every recursion loop.

The recursion shouldn't happen anymore, but it seems more robust to
reconnect after the connection has been successfully closed, otherwise
we might end up requesting more connections than we ought (hopefully not
as dramatically as here anyway).

fix(pool): fix infinite loop with close_returns=True

Close #1124

docs(pool): a proto-only SQLAlchemy url suffices as we are passing a creator

docs(pool): consistent sync/async examples in SQLAlchemy NullPool example

docs(pool): add documentation for close_returns

feat(pool): add close_returns

Behaviour implemented via subclassing Psycopg 3.2.

Close #1046

feat(pool): add connection subclasses to enable putconn-on-close

test(pool): make sure that returned connection have the pool set

The flow where we set _pool = None or _pool = self is a bit complex.
Make sure we don't mess it up.

Merge branch 'perf/transpose-loading'

perf: load results by row rather than by column

In the past we were converting results to Python tuples proceeding
column by column. The rationale was that certain overhead such as
selecting the loader was to be paid only once per column, not once per
datum.

However this analysis was dismantled in #1163, see especially the comment at
https://github.com/psycopg/psycopg/pull/1163#issuecomment-3288921422
for some benchmark comparing various conversion strategies. In the end,
the simple row-by-row loading in a single function ends up being more
performing. Performance now surpasses the one of psycopg2.

See also #1155 for an initial analysis of performance regression.

A big thank you to Jörg Breitbart (@jerch) for this improvement!

Merge pull request #1150 from jerch/fix/binary-copy-rowlevel

fix buffer in binary copy

refactor: drop copy format code path with buffer creation

The path is unused and leads to more convoluted code.

fix(c): fix copy buffer corruption on dumping error

Restore the buffer so that if we trap the exception and continue in
Python the buffer will not be corrupted.

Improve on the #1147 fix.

test: add test to reveal the buffer corruption on dump error

Merge pull request #1158 from jerch/fix/set-types-text

respect set_types in c-version in text format

docs: mention issue #1153 fix in news file

fix: consistent error passing a row with mismatched size to copy

fix(c): respect the set_types() definitions in text format

Fix #1153

ci: enable tests on PR

They shouldn't be duplicated anymore if we develop in private forks of
the repo.

test: add test to reproduce set_types being ignored

It has been reported ignored in the C implementation in binary format. See #1153

test: longer sleep to test for idle timeout exception

The test fails semi-regularly on macOS (probably 5% of the times).

Merge pull request #1141 from wasix-org/fix/poll-rw

Fix polling for read+write

test: make AEvent.wait_timeout() return False on timeout

For better similarity to Event.wait().

fix: consider a connection closed in poll() only if it's not ready too

When closing the proxy and trying to communicate, poll returns state 25
on the connection, i.e. POLLIN | POLLERR | POLLHUP. The connection is
not closed though, so we might end up in weird state ahead (e.g. connection
stuck in ACTIVE).

Because it's suggested that the connection is readable, do read from it.
Likely a proper error will be raised downstream, but we will remain in
consistent state.

test: fix decorator to skip tests on crdb because of any db name accepted

fix(c): fix timeout calculation in select-based wait function

Because of the error, the timeout set to 0, resulting in a busy loop
instead of waiting.

Might fix #645.

ci: use working test server on macOS

macos-14 can use PostgreSQL 18. I saw a failure trying to use 17; not
sure if it's still there but 18 works fine, so bump.

macos-13 better says on 17. Trying to "brew update" it will pull a
version that will try to compile docbook and takes forever to install. I
understand that avoid an upgrade we get a version that has a binary
package ("bottled", in this deranged naming convention).

refactor: check for closed connection in wait functions

Catch and display errors in a more homogeneous way: if a wait function
finds a connection closed it will raise an OperationalError chained to
the OSError obtained from stat'ing the socket. Previously control would
have gone back to the generator with a read-ready state and it would
have failed on whatever libpq function would have touched the socket.

Fix the problem reported in #608 affecting the epoll wait function, for
which we opted to use the poll function instead (which, more simply,
made the closed fd easier to spot).

note that on Windows it's not possible to use os.fstat() to ckeck a
socket state, therefore make it no-op.

ci: test with PostgreSQL 18 final image

test: improve wait functions tests

Check for consistent behaviour and expected result. The test currently
pass on the Python module and highlight shortcoming in the C function.

Complete wait async test, generate sync version automatically.

See #1141

refactor(wait): disallow interval=None in wait functions

It is not used internally: we always use an interval to allow for
Ctrl-C. It causes ambiguity with 0, which might be a value useful for
polling.

fix: fix return values of wait_poll(), wait_epoll()

fix(c): fix polling for read+write

Merge branch 'bump-libs'

ci: add workflow to build and cache the libpq

This avoid to attempt building the libpq concurrently with every Python
version, which will make caching fail.