{"push_id":2179422871,"size":101,"distinct_size":47,"ref":"refs/heads/windowfunc","head":"4fcb02b50f4089813ef10fd3e8c2c3d49773742e","before":"3a982674cbbd8729bed999f318d645fa749035ab","commits":[{"sha":"a22deb9fe93ba493ca8c36baf2b7feecbec0e89d","author":{"name":"Heikki Linnakangas","email":"05628fb5af084483329a74d57058afe6d592a16d@pivotal.io"},"message":"Fix initialization of GPDB-specific AO reloptions.\n\nThe options should only be registered once. Surprisingly, the duplicates\nseem to be harmless, because everything worked, but the accumulation of\nmore and more reloptions was slowing down tests that created a lot of\ntables.","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/a22deb9fe93ba493ca8c36baf2b7feecbec0e89d"},{"sha":"fb32b601333053fd17b2de4e433fa14fff135e6d","author":{"name":"Tom Lane","email":"9ca635a44fdf5aa326758d8d245579b9ae84653c@sss.pgh.pa.us"},"message":"Add missing reset of need_initialization in reloptions code.\n\nThis resulted in useless extra work during every call of parseRelOptions,\nbut no bad effects other than that.  Noted by Alvaro.\n\n(Cherry-picked from PostgreSQL commit eb9954e362)","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/fb32b601333053fd17b2de4e433fa14fff135e6d"},{"sha":"8b60e44de8c328e95368a36d62d176004a6c6cab","author":{"name":"Heikki Linnakangas","email":"05628fb5af084483329a74d57058afe6d592a16d@pivotal.io"},"message":"Remove some code that was duplicated by a mis-merge.\n\nHarmless, but clearly useless and unintentional.","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/8b60e44de8c328e95368a36d62d176004a6c6cab"},{"sha":"2c1ace7aaba7154ffc862641099b7c25207aa9c5","author":{"name":"Ashwin Agrawal","email":"0b80898d20c115b5a17919c67f80b4a69754d0af@pivotal.io"},"message":"Fix unused variables warning in RelationCreateStorage()","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/2c1ace7aaba7154ffc862641099b7c25207aa9c5"},{"sha":"448386b5ed4c6709813d1cb55ce4b23d683aeaf8","author":{"name":"Jesse Zhang","email":"5e2fc7e57000747fbf2d33f76a3ff452cec131a2@gmail.com"},"message":"Update SELECT version() to show whether it is a 32 or 64-bit backend binary.\n\n(cherry picked from commit b32a290bdfa0a17787985ec20c6696e412f8edcd)","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/448386b5ed4c6709813d1cb55ce4b23d683aeaf8"},{"sha":"0fbeb918344708fecab80468a514f879f58b616f","author":{"name":"Jesse Zhang","email":"5e2fc7e57000747fbf2d33f76a3ff452cec131a2@gmail.com"},"message":"Defer PG_VERSION_STR and PG_VERSION_NUM\n\nWe already had the pointer size check in the upstream order in our\ngenesis open source commit 6b0e52be , but we seem to have missed the\nrest of the rearrangement from upstream.\n\nThis commit completes that cherry-pick, to reduce merge diff with\n(admittedly much later) commit 46328916 .\n\n> Doesn't seem like a good idea to be doing AC_CHECK_SIZEOF(void *) so much\n> earlier than all the other sizeof checks, and it certainly fails to follow\n> the order suggested at the file head.  Rearrange.\nAuthor: Tom Lane\nAdapted by Pivotal\n\n(cherry picked from commit 075ac80d72cf2669f4b94d3a6808b2477f7ea611)","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/0fbeb918344708fecab80468a514f879f58b616f"},{"sha":"4b0f052d3ea9ea0bbf1da154766a7a460ffd418b","author":{"name":"Jesse Zhang","email":"5e2fc7e57000747fbf2d33f76a3ff452cec131a2@gmail.com"},"message":"configure: Allow adding a custom string to PG_VERSION\n\n> This can be used to mark custom built binaries with an extra version\n> string such as a git describe identifier or distribution package release\n> version.\n\nFrom: Oskari Saarenmaa <os@ohmu.fi>\nOriginally commited by Peter Eisentraut, adapted by Pivotal.\n(cherry picked from commit 46328916eefc5f9eaf249518e96f68afcd35923b)","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/4b0f052d3ea9ea0bbf1da154766a7a460ffd418b"},{"sha":"5e8e3588fc5922603a4bacaf9a9ce9db37f5cbf0","author":{"name":"David Sharp","email":"c63bac5dc0b19ca3fd287165142f9443b160a40d@pivotal.io"},"message":"Travis: Use upgraded compiler gcc-6","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/5e8e3588fc5922603a4bacaf9a9ce9db37f5cbf0"},{"sha":"0c0948c7db951e9916a15cb94117734a0b56fc0e","author":{"name":"David Sharp","email":"c63bac5dc0b19ca3fd287165142f9443b160a40d@pivotal.io"},"message":"configure: Update ACX_PTHREAD macro to be warning-free\n\nUpdate the C code for ACX_PTHREAD from the autoconf-archive\n\nThe relevant changes come from these commits:\n- http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=commitdiff;h=0728dab771b2a4da8032d5f0ade8870add680b39\n- http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=commitdiff;h=7a6c511965489c38f5d3bef4afe113e3fe11c2ac","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/0c0948c7db951e9916a15cb94117734a0b56fc0e"},{"sha":"d81079db6796809009a0885edd195aec7be005dd","author":{"name":"David Sharp","email":"c63bac5dc0b19ca3fd287165142f9443b160a40d@pivotal.io"},"message":"configure: Fix uninitialized warnings in c-library.m4\n\nFixes for:\n- PGAC_FUNC_GETTIMOFDAY_1ARG and\n- PGAC_FUNC_GETPWUID_R_5ARG","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/d81079db6796809009a0885edd195aec7be005dd"},{"sha":"10cc61083c8d1aa182b7f92278c547f6a8ebcfef","author":{"name":"David Sharp","email":"c63bac5dc0b19ca3fd287165142f9443b160a40d@pivotal.io"},"message":"walrep: Fix maybe-unitialized warnings in test_receive_and_verify()\n\nPrior to this commit, the below warnings were occurring on GCC 6.2.0.\n\nThe warnings seem valid but unlikely, so we just NULL initialize\nlogStreamStart, which would most likely still land in the correct error\ncase.\n\ngcc -O3 -std=gnu99  -Wall -Wmissing-prototypes -Wpointer-arith -Werror=uninitialized -Wendif-labels -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -fno-aggressive-loop-optimizations -Wno-unused-but-set-variable -Wno-address  -fpic -I. -I. -I../../../src/include -D_GNU_SOURCE -I/usr/include/libxml2   -c -o gplibpq.o gplibpq.c\nIn file included from ../../../src/include/postgres.h:53:0,\n                 from gplibpq.c:7:\ngplibpq.c: In function 'test_receive_and_verify':\n../../../src/include/utils/elog.h:311:3: error: 'logStreamStart.xrecoff' may be used uninitialized in this function [-Werror=maybe-uninitialized]\n   elog_finish(elevel, __VA_ARGS__); \\\n   ^~~~~~~~~~~\ngplibpq.c:140:15: note: 'logStreamStart.xrecoff' was declared here\n    XLogRecPtr logStreamStart;\n               ^~~~~~~~~~~~~~\nIn file included from ../../../src/include/postgres.h:53:0,\n                 from gplibpq.c:7:\n../../../src/include/utils/elog.h:311:3: error: 'logStreamStart.xlogid' may be used uninitialized in this function [-Werror=maybe-uninitialized]\n   elog_finish(elevel, __VA_ARGS__); \\\n   ^~~~~~~~~~~\ngplibpq.c:140:15: note: 'logStreamStart.xlogid' was declared here\n    XLogRecPtr logStreamStart;\n               ^~~~~~~~~~~~~~\n\nSigned-off-by: Amil Khanzada <akhanzada@pivotal.io>","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/10cc61083c8d1aa182b7f92278c547f6a8ebcfef"},{"sha":"4ee9323b5680697d40e66ddbf5290af2b39f446f","author":{"name":"David Sharp","email":"c63bac5dc0b19ca3fd287165142f9443b160a40d@pivotal.io"},"message":"configure: Enable -Werror=uninitialized","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/4ee9323b5680697d40e66ddbf5290af2b39f446f"},{"sha":"999359d051d1d8bb295c567671f033f7b2fb6677","author":{"name":"Amil Khanzada","email":"4ef8f786c2788b33756120303921fc8b1550c1af@pivotal.io"},"message":"dbcommands: fix maybe-unitialized warning from upstream\n\nPrior to this commit, the below warnings were occurring with GCC versions 6.2.0\non CentOS 6 and 6.3.0 on Ubuntu 14.\nThese are actually invalid because the `get_dbinfo()` call will always fill in\nthose variables but the compiler is not smart enough to detect this.\n\nThe code change of this commit has not happened on the latest upstream Postgres\n(10.1), so there will be merge conflicts in future, but we decided as a team\nin #3871 that it was better to fix the warnings.\n\nIn file included from ../../../src/include/postgres.h:53:0,\n                 from dbcommands.c:22:\ndbcommands.c: In function ‘createdb’:\n../../../src/include/utils/elog.h:184:4: error: ‘src_ctype’ may be used uninitialized in this function [-Werror=maybe-uninitialized]\n    errfinish rest; \\\n    ^~~~~~~~~\ndbcommands.c:616:11: note: ‘src_ctype’ was declared here\n  char    *src_ctype;\n           ^~~~~~~~~\nIn file included from ../../../src/include/postgres.h:53:0,\n                 from dbcommands.c:22:\n../../../src/include/utils/elog.h:184:4: error: ‘src_collate’ may be used uninitialized in this function [-Werror=maybe-uninitialized]\n    errfinish rest; \\\n    ^~~~~~~~~\ndbcommands.c:615:11: note: ‘src_collate’ was declared here\n  char    *src_collate;\n           ^~~~~~~~~~~\n\nSigned-off-by: David Sharp <dsharp@pivotal.io>","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/999359d051d1d8bb295c567671f033f7b2fb6677"},{"sha":"c7c5b6d2635b819b912219338016cca5eda82e11","author":{"name":"Jimmy Yih","email":"98e833c712dc566889d0111a71a89f7346ada3fc@pivotal.io"},"message":"Only allow fts probe process to touch fts probe pause GUC\n\nSome master processes were calling gpvars_assign_gp_fts_probe_pause\nwhen they should not. This was found when the new stats collecter\nprocess errored out on this function call. We also move it to fts.c\nwhere it will live alongside the other fts code.\n\nAlso, check for interrupts in fts sleep loop so that fts probe process\ncan process SIGHUP properly.","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/c7c5b6d2635b819b912219338016cca5eda82e11"},{"sha":"2295c1fdcb85b029469b0783f6ff9d5d11c5864b","author":{"name":"Jimmy Yih","email":"98e833c712dc566889d0111a71a89f7346ada3fc@pivotal.io"},"message":"isolation2 enhancement and fixes\n\n1. When forked command is not blocking, give the output that the pipe\n   received to see why it did not block.\n2. Fix utility mode quit so that a utility mode connection can be\n   restarted when connection is closed by postmaster.\n3. Fix ! shell commands by processing -- as part of the shell command\n   instead of the start of a SQL comment","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/2295c1fdcb85b029469b0783f6ff9d5d11c5864b"},{"sha":"e2d8afa92ceeca3bba2f40260fd98039957d4c83","author":{"name":"Jimmy Yih","email":"98e833c712dc566889d0111a71a89f7346ada3fc@pivotal.io"},"message":"fix mocker.py for mocking guc_gp.c\n\nThere was a hack to exclude guc.c from doing single-line comment (//)\nremoval when creating mock c files. The hack is no longer needed in\nguc.c but is now needed in guc_gp.c since the offending line was moved\nto guc_gp.c.","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/e2d8afa92ceeca3bba2f40260fd98039957d4c83"},{"sha":"08f4e145c6351d17e395d1827a0ef07796ddc7ac","author":{"name":"Jimmy Yih","email":"98e833c712dc566889d0111a71a89f7346ada3fc@pivotal.io"},"message":"Restrain replay of MMXLOG and smgr xlog records\n\nMMXLOG records should only be replayed in standby mode and smgr create\nxlog records should not do anything until persistent tables and MMXLOG\nrecords are removed from Greenplum.\n\nAuthor: Jimmy Yih <jyih@pivotal.io>\nAuthor: Asim Praveen <apraveen@pivotal.io>","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/08f4e145c6351d17e395d1827a0ef07796ddc7ac"},{"sha":"13889e3290d051abe76ac6fb3448a930a18e5f3a","author":{"name":"Jimmy Yih","email":"98e833c712dc566889d0111a71a89f7346ada3fc@pivotal.io"},"message":"Enable synchronous WAL replication when primary first detects mirror up\n\nWhen a primary segment first detects mirror is up, primary will\nautomatically turn on synchronous WAL replication by directly setting\nWalSndCtl->sync_standbys_defined to true during FTS probe handling\n(bypassing the need to have postmaster reload and Checkpointer process\ndoing the update). The state will be persisted by inserting\nsynchronous_standby_names=\"*\" into gp_replication.conf in case of\nsegment crash.","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/13889e3290d051abe76ac6fb3448a930a18e5f3a"},{"sha":"b2c636d6f1ea232cb83bc7637ffa71b431921b9b","author":{"name":"Jimmy Yih","email":"98e833c712dc566889d0111a71a89f7346ada3fc@pivotal.io"},"message":"Add to isolation2 .gitignore files\n\nThese were missed out and need to be ignored.\n\nAuthor: Jimmy Yih <jyih@pivotal.io>\nAuthor: Asim R Praveen <apraveen@pivotal.io>","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/b2c636d6f1ea232cb83bc7637ffa71b431921b9b"},{"sha":"44f8f90f09cef90a458a232001e87f854fd841be","author":{"name":"Abhijit Subramanya","email":"1cfaa38b071fee7ff57902436204673c4e1308a3@pivotal.io"},"message":"Remove non deterministic tests\n\nThe uao drop_while_vacuum tests are non deterministic they way they are\ncurrently written. For now we want to remove these tests to make the CI more\nstable. The tests will be replaced with a more deterministic version later.\n\nAuthor: Abhijit Subramanya <asubramanya@pivotal.io>\nAuthor: Taylor Vesely <tvesely@pivotal.io>","distinct":false,"url":"https://api.github.com/repos/vraghavan78/gpdb/commits/44f8f90f09cef90a458a232001e87f854fd841be"}]}
{"actor":{"display_login":"vraghavan78"}}