[pgpool-hackers: 209] Re: Ideas for code cleanup

Bruce Momjian bruce at momjian.us
Fri Apr 12 10:55:41 JST 2013


On Fri, Apr 12, 2013 at 09:32:54AM +0900, Tatsuo Ishii wrote:
> > On Fri, Apr 12, 2013 at 08:40:02AM +0900, Tatsuo Ishii wrote:
> >> > On Thu, Apr 11, 2013 at 11:42:59AM -0400, Bruce Momjian wrote:
> >> >> > > The goal is to allow the code to be more easily understood by new
> >> >> > > developers.  Would you like to see a patch?
> >> >> > 
> >> >> > Yes, please. Maybe you want to start from breaking main.c into several
> >> >> > files?
> >> >> 
> >> >> Sure. I will do that now and post a patch.
> >> > 
> >> > I started by just moving the pid/status-file processing functions to a
> >> > separate C file, so you could see how I am approaching it.  Patch
> >> > attached.
> >> 
> >> Thakns. I briefly looked into this and noticed that you modified
> >> Makefile.in.  You should modify Makefile.am because Makefile.in is
> >> generated automatically from Makefile.am.
> > 
> > Good catch.  Updated patch attached.
> 
> Thanks. Compiled without problem and I can start pgpool-II.
> 
> BTW, we are planning to remove "old"(non memory based) query cache in
> the near future. I think you can do it in the process of refactoring.

Sure, no problem.  I expect the refactoring to take weeks or months.

-- 
  Bruce Momjian  <bruce at momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + It's impossible for everything to be true. +


More information about the pgpool-hackers mailing list