[pgpool-hackers: 314] Re: reorganisation of pgpool-II source tree
Ahsan Hadi
ahsan.hadi at enterprisedb.com
Mon Jul 22 15:34:02 JST 2013
Thanks Tatsuo. When will you begin committing changes for 3.4?
Usama has made some improvements to the regression test suite for pgpool,
we will sending those change early this week.
Btw, I have added another task to the internally tracked todo list for
pgpool, it is
"We would like to see pgppol in separate sub modules that can be installed
separately for each feature, along with a common -libs subpackage or so.
The idea is that if someone just want to install load balancing for pgpool,
they should be able to do that without installing all the other features. I
suppose the changes done to the directory structure are a step in that
direction."
I am hoping we can also do this for 3.4.
-- Ahsan
On Mon, Jul 22, 2013 at 5:15 AM, Tatsuo Ishii <ishii at postgresql.org> wrote:
> Ahsan,
>
> Yes, now it looks good. If there's no objection, I'm looking forward
> to committing Usama's changes in 3.4.
> --
> Tatsuo Ishii
> SRA OSS, Inc. Japan
> English: http://www.sraoss.co.jp/index_en.php
> Japanese: http://www.sraoss.co.jp
>
> > Hi Tatsuo,
> >
> > Assuming there are no issues when are you planning to commit the
> directory
> > structure changes? I believe we would like these to be part of pgpool
> 3.4.
> >
> > -- Ahsan
> >
> >
> >
> > On Mon, Jul 15, 2013 at 5:09 PM, Muhammad Usama <m.usama at gmail.com>
> wrote:
> >
> >> Hi Tatsuo
> >>
> >> Thanks for your feedback, I have made the suggested changes and pushed
> to
> >> the github repository.
> >>
> >> --Usama
> >>
> >>
> >> On Thu, Jul 11, 2013 at 3:39 AM, Tatsuo Ishii <ishii at postgresql.org
> >wrote:
> >>
> >>> Usama,
> >>>
> >>> Thanks for the proposal. In over all I believe this will greatly help
> >>> people who are trying to understand the pgpool-II source code. After
> >>> pgpool-II 3.3 is out (which is expected in this July), we could apply
> >>> the changes if there's no objection from pgpool-II hackers.
> >>>
> >>> > Since the pgpool-II code base has grown in recent times and the code
> >>> tree
> >>> > is not very much organised into directories i.e There are
> >>> > no separate directories for source and include files, and most of the
> >>> pgpool-II
> >>> > source resides in top level pgpool2 directory, This makes it a little
> >>> hard
> >>> > to get the quick understanding of the code for new developers,
> >>> > and maintenance of code will become more and more difficult with
> growth
> >>> in
> >>> > code base.
> >>> >
> >>> > So me and Tatsuo has been working on the reorganisation of pgpool-II
> >>> source
> >>> > tree and have came up with the directory structure.
> >>> > The purposed source tree for the pgpool-II is to contains following
> >>> > directories.
> >>>
> >>> Here are small suggestions.
> >>>
> >>> I think you missed to create src/include/query_cache and you should
> >>> move src/include/utils/pool_memqcache.h there.
> >>>
> >>> Also I think src/rewrite/parallel_query should not under src/rewrite
> >>> because staffs under parallel_query are not for general query
> >>> rewriting, but for parallel_query. They look similar in implementation
> >>> but their aims are completely different. If you do that way, it will
> >>> give enough confusion to those who are studying pgpool-II source code.
> >>> --
> >>> Tatsuo Ishii
> >>> SRA OSS, Inc. Japan
> >>> English: http://www.sraoss.co.jp/index_en.php
> >>> Japanese: http://www.sraoss.co.jp
> >>>
> >>> > pgpool2
> >>> > | --doc
> >>> > | --m4
> >>> > | --pgpool_adm
> >>> > | --src
> >>> > | | --auth
> >>> > | | --config
> >>> > | | --context
> >>> > | | --include
> >>> > | | | --auth
> >>> > | | | --context
> >>> > | | | --parser
> >>> > | | | --pcp
> >>> > | | | --protocol
> >>> > | | | --rewrite
> >>> > | | | --utils
> >>> > | | | --watchdog
> >>> > | | --libs
> >>> > | | | --pcp
> >>> > | | --main
> >>> > | | --parser
> >>> > | | --pcp_con
> >>> > | | --protocol
> >>> > | | --query_cache
> >>> > | | --redhat
> >>> > | | | --rpm_installer
> >>> > | | --rewrite
> >>> > | | | --parallel_query
> >>> > | | --sample
> >>> > | | --sql
> >>> > | | | --pgpool-recovery
> >>> > | | | --pgpool-regclass
> >>> > | | --streaming_replication
> >>> > | | --system_db
> >>> > | | --test
> >>> > | | | --C
> >>> > | | | --jdbc
> >>> > | | | | --expected
> >>> > | | | --parser
> >>> > | | | | --expected
> >>> > | | | | --input
> >>> > | | | --pdo-test
> >>> > | | | | --mod
> >>> > | | | | --SQLlist
> >>> > | | | --regression
> >>> > | | | | --tests
> >>> > | | | | | --001.load_balance
> >>> > | | | | | --002.native_replication
> >>> > | | | | | --003.failover
> >>> > | | | | | --050.bug58
> >>> > | | | | | --051.bug60
> >>> > | | | | | --052.do_query
> >>> > | | | | | --053.insert_lock_hangs
> >>> > | | | --timestamp
> >>> > | | | | --expected
> >>> > | | | | --input
> >>> > | | --tools
> >>> > | | | --pcp
> >>> > | | | --pgmd5
> >>> > | | --utils
> >>> > | | --watchdog
> >>> > | | | --test
> >>> >
> >>> >
> >>> > I have created a git repository on github (
> >>> https://github.com/codeforall
> >>> > /pgpool2.git) with the purposed directory structure.
> >>> > and the attached (pgpool2_code_tree.txt) file contains the complete
> list
> >>> > directories along with files for the purposed directory structure.
> >>> >
> >>> > Thanks
> >>> >
> >>> > Regards
> >>> > Muhammad Usama
> >>>
> >>
> >>
> >> _______________________________________________
> >> pgpool-hackers mailing list
> >> pgpool-hackers at pgpool.net
> >> http://www.pgpool.net/mailman/listinfo/pgpool-hackers
> >>
> >>
> >
> >
> > --
> > Ahsan Hadi
> > Snr Director Product Development
> > EnterpriseDB Corporation
> > The Enterprise Postgres Company
> >
> > Phone: +92-51-8358874
> > Mobile: +92-333-5162114
> >
> > Website: www.enterprisedb.com
> > EnterpriseDB Blog: http://blogs.enterprisedb.com/
> > Follow us on Twitter: http://www.twitter.com/enterprisedb
> >
> > This e-mail message (and any attachment) is intended for the use of the
> > individual or entity to whom it is addressed. This message contains
> > information from EnterpriseDB Corporation that may be privileged,
> > confidential, or exempt from disclosure under applicable law. If you are
> > not the intended recipient or authorized to receive this for the intended
> > recipient, any use, dissemination, distribution, retention, archiving, or
> > copying of this communication is strictly prohibited. If you have
> received
> > this e-mail in error, please notify the sender immediately by reply
> e-mail
> > and delete this message.
>
--
Ahsan Hadi
Snr Director Product Development
EnterpriseDB Corporation
The Enterprise Postgres Company
Phone: +92-51-8358874
Mobile: +92-333-5162114
Website: www.enterprisedb.com
EnterpriseDB Blog: http://blogs.enterprisedb.com/
Follow us on Twitter: http://www.twitter.com/enterprisedb
This e-mail message (and any attachment) is intended for the use of the
individual or entity to whom it is addressed. This message contains
information from EnterpriseDB Corporation that may be privileged,
confidential, or exempt from disclosure under applicable law. If you are
not the intended recipient or authorized to receive this for the intended
recipient, any use, dissemination, distribution, retention, archiving, or
copying of this communication is strictly prohibited. If you have received
this e-mail in error, please notify the sender immediately by reply e-mail
and delete this message.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.sraoss.jp/pipermail/pgpool-hackers/attachments/20130722/24467588/attachment.html>
More information about the pgpool-hackers
mailing list