Move Windows test to separate file to avoid applying the asyncio mark

Make synchronous test async to avoid asyncio error

Configure codespell to run on a dirty working directory

Add doc section about server messages handling

Fix typo/improve errors docs

Fix docs typo

Note Python 3.6 dropped support in pool release notes

Prepare psycopg_pool 3.1 for release

Fix typo in prepared statements+pooling warning

Fix pool docs typo

Merge branch 'black-stable'

Apply string manipulations from black --preview

They are stable under the current black and look an improvement.

Restore black max line length to its default

Trying black --preview, it performs several aggressive changes that are
made worse by the stricter margin, so relax it to the default.

Move to using the stable black version 22.1.0

fix: confusing named parameters in example query

In the named parameters example query, the name of the parameters are
Python built in data types: int and str.

This is confusing because it looks like we are casting values passed
as arguments instead of passing named arguments.

Fix typo

Toilets are not to be taken lightly.

Typo fixed

Merge pull request #211 from The-One-And-Only-H/codespell

Add codespell to pipeline

Configure codespell in tox.ini

Fix spelling errors found by codespell

Rename noe function to no_e

abc.rst: Fix typos discovered by codespell

https://pypi.org/project/codespell/

Allow one test to fail on Windows C implementation

This test doesn't run in tox, so it doesn't benefit from repeats, and it
fails often indeed on flakey tests.

Add pytest option to tolerate a number of failures, and use it.

Don't reset the random seed re-running failed tests

We don't want to make them pass because we didn't like the random draw.
It's not best of three.

Drop dataclasses package from constraints file

It is part of the stdlib in all supported Python versions, so it is no
more an external dependency.

Add compatibility macro for PQsetTraceFlags

Avoid a warning of implicit declaration compiling with libpq < 14.

Fix definitions for pipeline replacements macros

The previous definition caused a warning for implicit conversion from
void* to int.

Allow passing the PIP_CONSTRAINT env var to tox

This allows to run tests using the oldest, rather than the newest,
dependency packages, by passing it the tests/constraints.txt file.

Use wheel 0.37 as minimum version

It is the first version officially supporting Python 3.10
https://wheel.readthedocs.io/en/stable/news.html

Drop explicit tests retries

Now we just retry all failing tests automatically.

Retry automatically flakey tests in tox runs

Bump min version required for pytest-asyncio to 0.17.0

Older versions fail for the presence of asyncio_mode in pytest config,
which is required from 0.17 on instead.

Configure the async loop at test session start

Doing it in a fixture is too late: event_loop might have already been
called if the first test running is async, and it would fail on Windows.

This started to be needed after pytest-asyncio 0.17.0 was released.
See <https://github.com/pytest-dev/pytest-asyncio/issues/256>.

Set asyncio mode in pytest configuration

Remove the warning raised in pytest-async >= 0.17.0 if the setting is
missing.

Merge branch 'null-pool'

More robust null pool tests

Fix race conditions of tasks completing before a concurrent client is
queued.

Fix test relying on fixed broken behaviour of copy().

Use connection.info rather than connection.pgconn to obtain higher level
results (e.g. Enum instead of int).

Refactor pools to reduce code duplication

Use common getconn/putconn implementations and implement only the
different behaviours in subclasses.

Add null-pools documentation

Add NullPool and AsyncNullPool

Close #148

Minor changes to pool docs

Improve pool docs, especially pools and connections life cycle

Add wait parameter to pool.open()

Relax some timing tests tolerances

Seeing them failing regularly in Github workflows.

Drop support for Python 3.6

Bump version numbers for release

Leave start year only in docs footer

Update, once and for good, the copyright year

After getting more information about the matter and having convinced
myself that there is no need, except FOMO, to extend the copyright year
on source code, change all our entries to leave only the start year.

    git ls-tree -r HEAD --name-only \
        | xargs sed -i '/Copyright.*\(Varrazzo\|Psycopg\)/ s/-20..//'

Don't leave the connection ACTIVE on error in COPY_OUT

Cancel the active COPY operation if the server has not finished sending
the data yet.

Close #203.

Also fix the tests which were based on this broken behaviour. A case of
self-administered Hyrum's law.

Use info instead of pgconn in copy tests

It displays better information in case of assert failed.

Add prepare_threshold connection parameter

This can help configuring connections to use PgBouncer. It is possible
to use the attribute in the pool kwargs, for instance, instead of using
the more complex configure callback.

Close #200.

Exclude mypy 0.931 too from tests

It seems still affected by https://github.com/python/mypy/issues/11820.

Don't overwrite the module of the psycopg.pq.pq_ctypes objects

It might be useful to tell apart the C version from the python versions
for debugging. The C versions don't do it, so they repr as:

    <psycopg_binary.pq.PGconn [IDLE] (database=piro) at 0x7f8556deb040>

See #201.

Add example about DDL statement with quoted parameter

See #199.

Add mark for pool tests

Don't auto-skip pool test if import fails. This would miss serious
problems leading to the pool not being importable. If someone wants to
skip the pool tests they can use `-m 'not pool'` now.

Drop broken test about setting prepare attributes on the class

I'm just clobbering the property here. It doesn't work. Thankfully it
wasn't documented to work...

Don't look up other modules objects in __del__ methods

The modules might have been already cleaned up during interpreted
shutdown. See <https://bugs.python.org/issue46256#msg409847> for an
explanation.

The stdlib guards against the same thing happening too. However they
take a reference in the function closure, which is stronger than what we
do. Doing so, on our strictly typed codebase, is a tedious chore, so, if
this is enough (it should be, according to the OP), I'm happy this way.

Close #198.

Allow pools to have min_size = 0 as long as they can grow

Add tests to verify they can grow from 0 no problem.

Add ConnectionTimeout subclass of OperationalError

To be used in the connection pool to detect timeout on connection.

Backported to Psycopg 3.0.8 to allow the pool 3.1 to work with it, at
least on diminished capacity (NullPool.connection() would time out only
for clients in the queue, not in case of new connection timeout).

Merge branch 'pool-checks'

Raise PoolClosed on wait() on a closed pool

Previously it might have raised an assert, if, for instance, wait would
have failed and retried.

Raise ValueError if the pool min_size is <= 0

Before it would have created a broken pool, blocking forever on getconn.

Move some common checks to the pool base class

Use caplog.set_level instead of manual filtering in tests

Skip tests using the deaf_port fixture on macOS and Windows

They fail some 50% of the time, looks like the port is not really
listening, the fact that the function is called listen() is just a
practical joke.

https://github.com/psycopg/psycopg/runs/4719288041?check_suite_focus=true

Add deaf_port fixture and base timeout tests on it

Merge branch 'pool-open'

Use the present tense in PoolClosed error messages

Thank you John Aldis! https://twitter.com/johnaldis/status/1478016200634425348

Report if the pool was closed, or never opened, in getconn() error message

Add API docs for pool open method/param.

AsyncConnectionPool.open() made async.

This makes it symmetrical with close(). However it doesn't really do any
async work as it's awkward to call it from init. Something we might do,
if that will be needed, could be to start the scheduler only and use it
to schedule immediately a call to an async _open().

In the future, an anyio-based pool might instead disallow open=True on
init.

See https://github.com/psycopg/psycopg/pull/151 for some discussion
about the topic.

Add open param to pool init

Disallow pool re-opening

There seems to be no use case for re-opening a closed pool, so disable
this now (until we find a real use-case).

Factor out a _stop_workers() method on connection pool classes

The method is symmetrical with open().
In particular, we take advantage of this method in
ConnectionPool.__del__() to remove some duplication.

Reset workers state in *ConnectionPool.close()

This way, the pools may be re-opened after a close, even if we'll
disallow this later on.

Add an open() method to connection pool classes

This method is responsible for setting the '_closed' attribute, which
hence now defaults to True in the base class, along with the
_sched_runner attribute, which is reset to None in close().

Remove occurrences of "thread" term in pool_async

There is no threading involved here, only async tasks. Adjust comments
and docstrings to avoid confusion.

Separate pool packages building into a different workflow

Psycopg and pool releases are separate.

Bump pool minor version to start adding features

Hopefully more robust test to identify a closed connection

Merge branch 'executemany-returning'

Add returning parameter on executemany()

Keep results from all queries run through executemany()

Instead of overwriting cursor's results at each execution in
executemany() we now accumulate the results of all queries on cursor
state by turning _set_results() into _extend_results(). This way, each
result remains accessible and one can use nextset() to move from one
result to another.

Drop unneeded method signatures in docs

Rename a couple of internal cursor methods for clarity

Refactor cur._raise_from_results() into _raise_for_result()

The case of raising with different statuses doesn't practically happen.

Drop Cursor._set_results()

Split it into a _check_results() and leave the caller manage the cursor state.

Shorten traceback on executemany

Fix typo in comment

Fix test suite to run on database in locales different from English

Note in news file that 3.0.8 is still unreleased

Fix spurious warning about variable referenced before assignment

Add flake8 configuration in other package dirs

Vim ALE is confused otherwise. I don't really get how is that it picks
the package directory as the starting one (maybe it finds the setup.py)?

Change type of `wait` in `pool_async.py`

Will become an incompatible type in assignment with the release of
https://github.com/python/typeshed/pull/6783

Fix py_codecs data type

It doesn't contain str anymore since 3.0.4.

A stricter definition revealed a couple of wrong usages of the mapping.
Use more consistently the higher level `conninfo_encoding()` function
instead of directly the mapping.

Use conninfo encoding to encode errors on connection

PostgreSQL returns connection errors in the encoding specified in
lc_messages (e.g. EUC-JP for a server with `lc_messages=ja_JP.EUC-JP`).
However, because we don't have a connection, the message was decoded in
the fallback utf-8.

If the client has specified a client_encoding, use it to decode the
error message. Be more lenient than usual to look up the encoding
(because it is not normalised by the server and we don't care about
performance as this only happens on error handling). As for any other
error messages, still use an error=replace policy to avoid exploding
reporting an error in the wrong encoding.

Close #194.

Fix typo: a raw data -> raw data

Fix doc link to transactions management

Also, "transactions", not "transaction".

Add note about installing libpq via psql

Merge branch 'dependencies-bounds'