test session mode and config reload for real (#33)

* test session mode and config reload for real

* period

* run them at the end

* typo

* wrong dir

Author: levkk

.circleci/run_tests.sh

@@ -34,9 +34,6 @@ psql -e -h 127.0.0.1 -p 6432 -f tests/sharding/query_routing_test_select.sql > /
 # Replica/primary selection & more sharding tests
 psql -e -h 127.0.0.1 -p 6432 -f tests/sharding/query_routing_test_primary_replica.sql > /dev/null
 
-# Test reload config
-kill -SIGHUP $(pgrep pgcat)
-
 #
 # ActiveRecord tests!
 #
@@ -45,6 +42,16 @@ sudo gem install bundler
 bundle install
 ruby tests.rb
 
+cd ../../
+# Test session mode (and config reload)
+sed -i 's/pool_mode = "transaction"/pool_mode = "session"/' pgcat.toml
+
+# Reload config
+kill -SIGHUP $(pgrep pgcat)
+
+# Prepared statements that will only work in session mode
+pgbench -h 127.0.0.1 -p 6432 -t 500 -c 2 --protocol prepared
+
 # Attempt clean shut down
 killall pgcat -s SIGINT
 

README.md

@@ -50,7 +50,7 @@ See [sharding README](./tests/sharding/README.md) for sharding logic testing.
 | **Feature**           | **Tested in CI**   | **Tested manually** | **Comments**                                                                                                             |
 |-----------------------|--------------------|---------------------|--------------------------------------------------------------------------------------------------------------------------|
 | Transaction pooling   | :heavy_check_mark: | :heavy_check_mark:  | Used by default for all tests.                                                                                           |
-| Session pooling       | :x:                | :heavy_check_mark:  | Easiest way to test is to enable it and run pgbench - results will be better than transaction pooling as expected.       |
+| Session pooling       | :heavy_check_mark: | :heavy_check_mark:  | Tested by running pgbench with `--protocol prepared` which only works in session mode.                                   |
 | `COPY`                | :heavy_check_mark: | :heavy_check_mark:  | `pgbench -i` uses `COPY`. `COPY FROM` is tested as well.                                                                 |
 | Query cancellation    | :heavy_check_mark: | :heavy_check_mark:  | `psql -c 'SELECT pg_sleep(1000);'` and press `Ctrl-C`.                                                                   |
 | Load balancing        | :x:                | :heavy_check_mark:  | We could test this by emitting statistics for each replica and compare them.                                             |