[COMMITTERS] pgsql: Improve generation algorithm for database system identifier.

2014-04-26 Thread Tom Lane
Improve generation algorithm for database system identifier. As noted some time ago, the original coding had a typo ("|" for "^") that made the result less unique than intended. Even the intended behavior is obsolete since it was based on wanting to produce a usable value even if we didn't have i

[COMMITTERS] pgsql: Don't #include utils/palloc.h in common/fe_memutils.h.

2014-04-26 Thread Tom Lane
Don't #include utils/palloc.h in common/fe_memutils.h. This breaks the principle that common/ ought not depend on anything in the server, not only code-wise but in the headers. The only arguable advantage is avoidance of duplication of half a dozen extern declarations, and even that is rather dub

[COMMITTERS] pgsql: Don't #include utils/palloc.h in common/fe_memutils.h.

2014-04-26 Thread Tom Lane
Don't #include utils/palloc.h in common/fe_memutils.h. This breaks the principle that common/ ought not depend on anything in the server, not only code-wise but in the headers. The only arguable advantage is avoidance of duplication of half a dozen extern declarations, and even that is rather dub

[COMMITTERS] pgsql: Record the proper typmod for an index expression column.

2014-04-26 Thread Tom Lane
Record the proper typmod for an index expression column. We should use exprTypmod() to extract the typmod of the expression, instead of just blindly storing -1. This seems to have been an aboriginal oversight in commit fc8d970cbcdd6f025475822a4cf01dfda0873226 which introduced general-expression i