fix(c): drop duplicate PQconsumeInput call

Also add a couple of C types in the function for better C code.

fix: use a correct recursive definition for DumperKey

Already supported in current Mypy: https://github.com/python/mypy/issues/731.

The definition of DumperKey was wrong anyway, in a way causing mypy to
crash: see https://github.com/python/mypy/issues/14000.

lint: fix type hints with disabled bytearray/memoryview/bytes equivalience

This changeset makes the code compatible with the current Mypy 0.981,
but passes most of the checks that Mypy 0.990 enforces if the byte
strings equivalence is disabled.

fix: define Escaping method as returning bytes

This change is backward compatible because bytes has a superset of the
memoryview interface and, in Python, to make pretty much any use of it,
it must be converted to bytes beforehand.

Close #422.

fix: fix check for empty notice handlers list

Issue reported by Mypy 0.990. Thank you!

docs: use consistenty `!this` to represent parameters and non-linked code

Don't use *this*, which is more of a math use but it disagrees with the
style rendered on the docs, and don't use ``this``, left to literals.

perf(c): only use __cinit__ in adaptation classes

There is less overhead creating the objects.

perf: avoid lookups of Wait and Ready enum values

docs: document how to unbreak editable mode in the readme

chore: bump to next dev version

docs: typographic love to the 2to3 page

docs: add section about returning multiple results

Close #417.

Merge branch 'build-macos-py311'

ci: re-enable suspended job after building the missing image

ci: configure in order to build only Python 3.11 packages for macOS x86_64

The required image was not available at the time of building other
packages. See #424.

chore: bump psycopg-pool version to 3.1.3

chore: bump to version number 3.1.4

chore: upgrade binary dependencies to libpq 15.0, openssl 1.1.1r

Merge branch 'py311'

ci: build Python 3.11 macOS arm64 packages

ci: skip test/build on macOS x86_64 for Python 3.11

Package not available yet. See #424

docs: mention Python 3.11 support in news

ci: build packages for Python 3.11 too

chore: add Python 3.11 support metadata

chore: test with Python 3.11 rc1

chore: upgrade Cython to 3.0a11 to support Python 3.11

chore: upgrade actions/upload-artifact version

test: print out the file to delete in order to clear the segfault indicator

test: terminate the session early if connection to the test db fails

The error might be masked by other failures. It's also not useful to
receive the same message hundreds of times.

Merge branch 'pg15'

ci: Use Postgres 15 images and packages after official release

docs: declare PostgreSQL 15 supported

chore: add errors defined in PostgreSQL 15

test: include PostgreSQL RC 1 and libpq 15 in the tests

chore: upgrade github actions

The currently used ones raise deprecation warnings.

test: document --pq-trace and --pq-debug pytest option

test: add --pq-debug pytest option to use PGconnDebug

fix: lock the connection during all stream() lifetime

Namely, also lock the connection when closing the generator, in the
finally clause, where we are fetching results.

refactor: handle exit exception in BasePipeline._exit()

We remove repeated code in __(a)exit__() method of (Async)Pipeline by
passing the exception value to _exit() method of BasePipeline.

chore: bump version number to next dev release

Merge pull request #401 from dlax/pipeline-no-extra-begin

fix: sync pipeline after implicit BEGIN

fix: sync pipeline after implicit BEGIN

Prior to this change, the following code:

    with conn.pipeline():
       conn.execute("select 'x'").fetchone()
       conn.execute("select 'y'")

would produce the following libpq trace:

    F 13 Parse  "" "BEGIN" 0
    F 14 Bind  "" "" 0 0 1 0
    F 6 Describe  P ""
    F 9 Execute  "" 0
    F 18 Parse  "" "select 'x'" 0
    F 14 Bind  "" "" 0 0 1 0
    F 6 Describe  P ""
    F 9 Execute  "" 0
    F 4 Flush
    B 4 ParseComplete
    B 4 BindComplete
    B 4 NoData
    B 10 CommandComplete  "BEGIN"
    B 4 ParseComplete
    B 4 BindComplete
    B 33 RowDescription  1 "?column?" NNNN 0 NNNN 65535 -1 0
    B 11 DataRow  1 1 'x'
    B 13 CommandComplete  "SELECT 1"
    F 13 Parse  "" "BEGIN" 0
    F 14 Bind  "" "" 0 0 1 0
    F 6 Describe  P ""
    F 9 Execute  "" 0
    F 18 Parse  "" "select 'y'" 0
    F 14 Bind  "" "" 0 0 1 0
    F 6 Describe  P ""
    F 9 Execute  "" 0
    B 4 ParseComplete
    B 4 BindComplete
    B 4 NoData
    B NN NoticeResponse  S "WARNING" V "WARNING" C "25001" M "there is already a transaction in progress" F "SSSS" L "SSSS" R "SSSS" \x00
    F 4 Sync
    B 10 CommandComplete  "BEGIN"
    B 4 ParseComplete
    B 4 BindComplete
    B 33 RowDescription  1 "?column?" NNNN 0 NNNN 65535 -1 0
    B 11 DataRow  1 1 'y'
    B 13 CommandComplete  "SELECT 1"
    B 5 ReadyForQuery  T

where we can see that the BEGIN statement (due to the connection being
in non-autocommit mode) is emitted twice, as notified by the server.

This is because the transaction state after the implicit BEGIN is not
"correct" (i.e. should be INTRANS, but is IDLE) since the result from
respective statement has not been received yet.

By syncing after the BEGIN, we fetch result from this command thus get
the transaction state INTRANS for following queries. This is similar to
what happens with explicit transaction, by using nested pipelines.

chore: bump to version number 3.1.3

Merge branch 'fix-399'

fix: fix handling of queries with %% in ClientCursor

Close #399.

#397: Fix typo in install.rst

Remove warning about missing binary package for M1

Binary packages for Apple M1 was added in #162.

Merge branch 'stream-close'

fix: make sure to terminate query on gen.close() from Cursor.stream()

Fix #382

perf: use functools.cache where available

Faster than lru_cache; use it when we don't suppose we will ever evict
values.

Merge branch 'fix-escape-nametuple-field'

docs: mention named tuples regression fix in change log

test: more exhaustive test of difficult names/attrs in composites

fix: restore pre-3.1 behaviour converting fields to named tuples attributes

Let more valid identifiers through _as_python_identifier(). Also 2.5x faster if no change is needed

Close #386

test: check unusual column names and namedtuple

fix: avoid using type variables in typevar bounds

It is not allowed by the specs. Mypy tolerates it, replacing the
argument with Any, according to @erictraut. Pyright reports it as an
error instead.

See #365, microsoft/pyright#3984

chore: upgrade mypy to 0.981

Most changes needed by https://github.com/python/typeshed/pull/8446.

docs: fix plural

docs: mention macOS arm64 packages distribution

build: fix starting Postgres in macOS build script

The brew command fails with:

    Could not enable service: 125: Domain does not support specified action
    Error: Failure while executing; `/bin/launchctl enable gui/501/homebrew.mxcl.postgresql@14` exited with 125.

docs: typo messge -> message

Thank you, codespell 💀💀💀

docs: add link to document how to recover the stream from ACTIVE state

See #382.

chore: bump to next dev version number

chore: bump to version number 3.1.2

fix: fix queries with repeated named parameters in ClientCursor

Close #378.

test: allow to trace libpq to STDERR, rename option to --pq-trace

By using STDERR, we get the trace directly in pytest output for failing
tests. Though this requires care when used in combination with
--capture=sys because this would make sys.stderr an in-memory file.

When STDERR is used, we don't insert the header since the test case
identifier will be interleaved already.

fix: also sync nested pipeline when a transaction is active

Upon enter, transaction() checks the in-transaction status to determine
if a BEGIN (when IDLE) or a SAVEPOINT (otherwise) should be inserted.
However, in pipeline mode, if the pipeline is in "implicit transaction"
(i.e. it has no active BEGIN) and statements have been sent, the
in-transaction status might be ACTIVE if those statements have not yet
completed (typically, when results have not been fetched).

By issuing a sync() before entering a nested pipeline (which will happen
when entering transaction(), if already within a pipeline), we force
completion of those statements, and thus get a predictable
in-transaction status before entering the transaction() block.

Closes #374.

test: fail test session in case of segfault

Before that, tox reruns would have made tests pass, because failed tests
wouldn't have gotten a chance to be written in the cache.

Merge branch 'fix-372'

fix(psycopg_c): propagate exceptions in Transformer.set_pgresult()

Fix #372

test: add test to verify the behavior of adapters failing init

An exception in loader init doesn't get raised and causes a segfault.
This reproduces the issue in #372.

Merge branch 'docs/pgconn'

docs: document PGconn's tracing methods

docs: document PGconn.encrypt_password()

Including an example of usage.

refactor(psycopg_c): define loggers at module level

fix(psycopg_c): add missing logger declaration

Merge branch 'fix-371'

docs: mention Windows timezone problem fixed in change log

fix: catch any type of error if ZoneInfo creation fails

OSError would be enough to solve the problem reported in #371. However,
in case of error, Windows C implementation segfaults. I would prefer to
not chase this error just because of weirdness in the tzdata external
library, so let's cast a wider net.

Fix #371

test: add bad timezone to simulate a reported timezone issue

The bad timezone causes OSError on Windows.

See #371

chore(pool): bump to next dev version number

chore: bump psycopg-pool version to 3.1.2

Merge branch 'fix-370'

fix(pool): continue growing if the pool is not full yet

Previously, starting from an empty pool, we would have stopped at the
first connection created.

fix(pool): allow growing again after failing a growing connection attempt

Fix #370

chore: bump to next dev version number

chore: bump to version number 3.1.1

Merge branch 'macos-broken'

docs: add news entry about Homebrew fix

refactor: add find_libpq_full_path() to use both in library and tests

fix(macos): find the libpq in non-standard places

chore: upgrade QEMU to v2 for build

Also make sure to use a version not affected by bug #304

fix: make sure to pass a port number to getaddrinfo

Was passing an empty string by mistake, if no port was specified. Mostly
not a problem, except on some platforms, where it causes an
"unrecognized service" error.

Close #366.

Bump to next dev release

docs: fix title level in news

chore: bump version number to release 3.1

chore: bump to version 3.0.17

Merge branch 'crdb-ci'

ci: run CRDB tests in CI

test(crdb): consider Decimal("Infinity") not supported in CRDB < 22