Skip to content

Commit

Permalink
Revert "Use new connection pool by default (#7706)"
Browse files Browse the repository at this point in the history
This reverts commit eda72c0.

There was a logic error in this commit, and so it did not actually
make the new connection pool the default.

I tried making it the default properly, and hit a number of new test
failures, so for now let's just clarify the situation.
  • Loading branch information
msullivan committed Oct 22, 2024
1 parent a131a2b commit a0dd107
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion edb/server/connpool/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
# During the transition period we allow for the pool to be swapped out. The
# current default is to use the old pool, however this will be switched to use
# the new pool once we've fully implemented all required features.
if os.environ.get("EDGEDB_USE_NEW_CONNPOOL", "") == "0":
if os.environ.get("EDGEDB_USE_NEW_CONNPOOL", "") == "1":
Pool = Pool2Impl
Pool2 = Pool1Impl
else:
Expand Down

0 comments on commit a0dd107

Please sign in to comment.