<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 3, 2016 at 4:51 PM, Tatsuo Ishii <span dir="ltr">&lt;<a href="mailto:ishii@postgresql.org" target="_blank">ishii@postgresql.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Usama,<br>
<br>
Now many of regression tests fail after this commit. Is it possible<br>
that you forgot to commit something?<br></blockquote><div><br></div><div>I am looking into same. Apparently its a memory issue. Hopefully I will be able to fix it today.</div><div><br></div><div>Best regards</div><div>Muhammad Usama</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Best regards,<br>
--<br>
Tatsuo Ishii<br>
SRA OSS, Inc. Japan<br>
English: <a href="http://www.sraoss.co.jp/index_en.php" rel="noreferrer" target="_blank">http://www.sraoss.co.jp/index_en.php</a><br>
Japanese:<a href="http://www.sraoss.co.jp" rel="noreferrer" target="_blank">http://www.sraoss.co.jp</a><br>
<div class="HOEnZb"><div class="h5"><br>
&gt; Streamlining the pgpool configuration system (Work in progress).<br>
&gt;<br>
&gt; This commit builds the framework for managing the pgpool-II configuration<br>
&gt; parameters. It is the first step towards implementing the session level<br>
&gt; SET command functionality in pgpool to allow the users to alter the pgpool<br>
&gt; config parameters for a particular session only and reset it back to the<br>
&gt; original value after the user session disconnects.<br>
&gt; The overall design of the configuration framework is inspired by the<br>
&gt; PostgreSQL&#39;s GUC and some of the more visible changes introduced by the<br>
&gt; framework are.<br>
&gt;<br>
&gt; -- There is no more a long list of if-else statements in pool_get_config function<br>
&gt; and now a simple loop can be used to alter/initialize/print all the pgpool-II<br>
&gt; config variables.<br>
&gt; -- The framework keeps track of values assigned to parameters in BOOT or INIT<br>
&gt; contexts and later value of the config parameters can be reset to that value.<br>
&gt;<br>
&gt; In addition to the framework, the commit also revisits the internal data types<br>
&gt; of the configuration parameters and made the following changes<br>
&gt; -- Changes the type of all on/off type config parameters from integer to boolean type<br>
&gt; -- All the string (char*) type parameters that can only be assigned from the<br>
&gt; particular set of values (e.g. master_slave_sub_mode can assign with either<br>
&gt; &quot;stream&quot; or &quot;Slony&quot;) are changed to Enum type parameters.<br>
&gt;<br>
&gt; Branch<br>
&gt; ------<br>
&gt; master<br>
&gt;<br>
&gt; Details<br>
&gt; -------<br>
&gt; <a href="http://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=414ffa4d4061f5eba82e36e4e455d2a0fa2b80d9" rel="noreferrer" target="_blank">http://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=414ffa4d4061f5eba82e36e4e455d2a0fa2b80d9</a><br>
&gt;<br>
&gt; Modified Files<br>
&gt; --------------<br>
&gt; src/Makefile.am                               |    1 +<br>
&gt; src/Makefile.in                               |   21 +-<br>
&gt; src/config/pool_config.c                      | 3666 ++-----------------------<br>
&gt; src/config/pool_config.l                      | 3623 ++----------------------<br>
&gt; src/config/pool_config_variables.c            | 3233 ++++++++++++++++++++++<br>
&gt; src/context/pool_query_context.c              |    4 +-<br>
&gt; src/include/pcp/libpcp_ext.h                  |    2 +-<br>
&gt; src/include/pool.h                            |    6 +-<br>
&gt; src/include/pool_config.h                     |  407 +--<br>
&gt; src/include/pool_config_variables.h           |  274 ++<br>
&gt; src/include/pool_type.h                       |   11 +<br>
&gt; src/include/utils/fe_ports.h                  |   11 +-<br>
&gt; src/include/utils/json.h                      |   14 +-<br>
&gt; src/include/utils/json_writer.h               |    2 +<br>
&gt; src/libs/pcp/Makefile.in                      |   16 +-<br>
&gt; src/main/main.c                               |    5 +-<br>
&gt; src/main/pgpool_main.c                        |   12 +-<br>
&gt; src/pcp_con/pcp_child.c                       |    2 +-<br>
&gt; src/pcp_con/pcp_worker.c                      |    8 +-<br>
&gt; src/protocol/child.c                          |    2 +-<br>
&gt; src/protocol/pool_process_query.c             |    2 +-<br>
&gt; src/query_cache/pool_memqcache.c              |   10 +-<br>
&gt; src/streaming_replication/pool_worker_child.c |    8 +-<br>
&gt; src/tools/pgmd5/Makefile.am                   |    3 +<br>
&gt; src/tools/pgmd5/Makefile.in                   |   26 +-<br>
&gt; src/tools/pgmd5/pg_md5.c                      |    2 +-<br>
&gt; src/tools/pgmd5/pool_config.c                 | 3499 ++---------------------<br>
&gt; src/utils/json.c                              |   20 +<br>
&gt; src/utils/json_writer.c                       |   11 +<br>
&gt; src/utils/pool_process_reporting.c            |    8 +-<br>
&gt; src/utils/regex_array.c                       |    2 +-<br>
&gt; src/watchdog/watchdog.c                       |    4 +-<br>
&gt; src/watchdog/wd_json_data.c                   |   59 +-<br>
&gt; src/watchdog/wd_lifecheck.c                   |   16 +-<br>
&gt; 34 files changed, 4611 insertions(+), 10379 deletions(-)<br>
&gt;<br>
&gt; _______________________________________________<br>
&gt; pgpool-committers mailing list<br>
&gt; <a href="mailto:pgpool-committers@pgpool.net">pgpool-committers@pgpool.net</a><br>
&gt; <a href="http://www.pgpool.net/mailman/listinfo/pgpool-committers" rel="noreferrer" target="_blank">http://www.pgpool.net/mailman/listinfo/pgpool-committers</a><br>
</div></div></blockquote></div><br></div></div>