partial header cleanup

Поиск
Список
Период
Сортировка
От Zdenek Kotala
Тема partial header cleanup
Дата
Msg-id 48402849.2050809@sun.com
обсуждение исходный текст
Ответы Re: partial header cleanup  (Alvaro Herrera <alvherre@commandprompt.com>)
Re: partial header cleanup  (Alvaro Herrera <alvherre@commandprompt.com>)
Список pgsql-patches
This replace xlog.h with xlogdefs.h in bufpage.h. All other changes are
forgotten include somewhere. It reduce e.g. bloat to half in itup.h. But, There
are still unresolved problems. htup should include bufpage.h, because it needs
PageHeader size, but there is still unnecessary bufmgr.h include in bufpage
which generates bloat.

See itup.h bloating:

http://doxygen.postgresql.org/itup_8h.html

this patch reduce xlog side. But there still about 18 unnecessary includes.



        Zdenek

PS: Thanks to Stefan K. He enabled graphs.
Index: src/backend/nodes/print.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/nodes/print.c,v
retrieving revision 1.87
diff -c -r1.87 print.c
*** src/backend/nodes/print.c    1 Jan 2008 19:45:50 -0000    1.87
--- src/backend/nodes/print.c    30 May 2008 15:13:42 -0000
***************
*** 20,25 ****
--- 20,26 ----
  #include "postgres.h"

  #include "access/printtup.h"
+ #include "lib/stringinfo.h"
  #include "nodes/print.h"
  #include "optimizer/clauses.h"
  #include "parser/parsetree.h"
Index: src/backend/postmaster/postmaster.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/postmaster/postmaster.c,v
retrieving revision 1.557
diff -c -r1.557 postmaster.c
*** src/backend/postmaster/postmaster.c    4 May 2008 21:13:35 -0000    1.557
--- src/backend/postmaster/postmaster.c    30 May 2008 15:13:42 -0000
***************
*** 93,98 ****
--- 93,99 ----
  #endif

  #include "access/transam.h"
+ #include "access/xlog.h"
  #include "bootstrap/bootstrap.h"
  #include "catalog/pg_control.h"
  #include "lib/dllist.h"
Index: src/backend/utils/adt/domains.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/utils/adt/domains.c,v
retrieving revision 1.6
diff -c -r1.6 domains.c
*** src/backend/utils/adt/domains.c    1 Jan 2008 19:45:52 -0000    1.6
--- src/backend/utils/adt/domains.c    30 May 2008 15:13:42 -0000
***************
*** 33,38 ****
--- 33,39 ----

  #include "commands/typecmds.h"
  #include "executor/executor.h"
+ #include "lib/stringinfo.h"
  #include "utils/builtins.h"
  #include "utils/lsyscache.h"

Index: src/backend/utils/fmgr/fmgr.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/utils/fmgr/fmgr.c,v
retrieving revision 1.119
diff -c -r1.119 fmgr.c
*** src/backend/utils/fmgr/fmgr.c    15 May 2008 00:17:40 -0000    1.119
--- src/backend/utils/fmgr/fmgr.c    30 May 2008 15:13:42 -0000
***************
*** 19,24 ****
--- 19,25 ----
  #include "catalog/pg_language.h"
  #include "catalog/pg_proc.h"
  #include "executor/functions.h"
+ #include "lib/stringinfo.h"
  #include "miscadmin.h"
  #include "parser/parse_expr.h"
  #include "pgstat.h"
Index: src/include/access/gin.h
===================================================================
RCS file: /projects/cvsroot/pgsql/src/include/access/gin.h,v
retrieving revision 1.20
diff -c -r1.20 gin.h
*** src/include/access/gin.h    16 May 2008 16:31:01 -0000    1.20
--- src/include/access/gin.h    30 May 2008 15:13:43 -0000
***************
*** 14,19 ****
--- 14,20 ----

  #include "access/itup.h"
  #include "access/relscan.h"
+ #include "access/xlog.h"
  #include "fmgr.h"
  #include "nodes/tidbitmap.h"
  #include "storage/block.h"
Index: src/include/access/heapam.h
===================================================================
RCS file: /projects/cvsroot/pgsql/src/include/access/heapam.h,v
retrieving revision 1.134
diff -c -r1.134 heapam.h
*** src/include/access/heapam.h    12 May 2008 00:00:53 -0000    1.134
--- src/include/access/heapam.h    30 May 2008 15:13:43 -0000
***************
*** 17,22 ****
--- 17,23 ----
  #include "access/htup.h"
  #include "access/relscan.h"
  #include "access/sdir.h"
+ #include "access/xlog.h"
  #include "nodes/primnodes.h"
  #include "storage/lock.h"
  #include "utils/snapshot.h"
Index: src/include/access/nbtree.h
===================================================================
RCS file: /projects/cvsroot/pgsql/src/include/access/nbtree.h,v
retrieving revision 1.118
diff -c -r1.118 nbtree.h
*** src/include/access/nbtree.h    16 Apr 2008 23:59:40 -0000    1.118
--- src/include/access/nbtree.h    30 May 2008 15:13:43 -0000
***************
*** 17,22 ****
--- 17,23 ----
  #include "access/itup.h"
  #include "access/relscan.h"
  #include "access/sdir.h"
+ #include "access/xlog.h"
  #include "access/xlogutils.h"


Index: src/include/storage/bufpage.h
===================================================================
RCS file: /projects/cvsroot/pgsql/src/include/storage/bufpage.h,v
retrieving revision 1.79
diff -c -r1.79 bufpage.h
*** src/include/storage/bufpage.h    12 May 2008 16:06:10 -0000    1.79
--- src/include/storage/bufpage.h    30 May 2008 15:13:43 -0000
***************
*** 14,20 ****
  #ifndef BUFPAGE_H
  #define BUFPAGE_H

! #include "access/xlog.h"
  #include "storage/bufmgr.h"
  #include "storage/item.h"
  #include "storage/off.h"
--- 14,20 ----
  #ifndef BUFPAGE_H
  #define BUFPAGE_H

! #include "access/xlogdefs.h"
  #include "storage/bufmgr.h"
  #include "storage/item.h"
  #include "storage/off.h"

В списке pgsql-patches по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: minor change to replace function comment
Следующее
От: Teodor Sigaev
Дата:
Сообщение: GIN improvements