- 05 11月, 2015 3 次提交
-
-
由 Heikki Linnakangas 提交于
-
由 Haisheng Yuan 提交于
When user runs \d or \d+ on a partition table, it will display partition keys on the current level for the partition table with the clause of 'Partition by: (partition_key_1, ..., partition_key_n)' after 'Distributed by' clause. But if user runs \d or \d+ on a non-partition table or leaf level partition table, the 'Partition by' clause will not be displayed. Haisheng Yuan Closes #25
-
由 Ashwin Agrawal 提交于
Relation can potential be dropped while waiting to acquire lock. If cannot find the relation for "if_exists" case should just emit NOTICE and act as noop.
-
- 04 11月, 2015 5 次提交
-
-
由 Entong Shen 提交于
Exchanging a default partition is currently not allowed because there is no validation on the data being exchanged. This commit add a GUC to enable this if the user chooses to do so.
-
由 Chumki Roy 提交于
Added to easily find the gptransfer work directory.
-
由 Jimmy Yih 提交于
Fix gpssh issues by upgrading pexpect/pxssh to latest stable version 3.3. Cherrypicked pxssh v4.0 ssh options into this upgrade too. Made ssh_utils.py pxssh initial connection validations threaded to reduce overhead of new pxssh validation function. The issue is pxssh and ssh related: http://pexpect.readthedocs.org/en/latest/commonissues.html#timing-issue-with-send-and-sendline
-
由 Entong Shen 提交于
-
由 Jimmy Yih 提交于
Refactor gpcrondump and gpdbrestore utilities (along with supporting dump.py, restore.py, and backup_utils.py from gppylib) to remove DUMP_DIR and DUMP_PREFIX global variables.
-
- 03 11月, 2015 11 次提交
-
-
由 Ashwin Agrawal 提交于
Drop on segments was performed always using if exists, this may cause inconsistency in cluster if for some reason table is found on master but misses to find on segments. Patch now dispatchs DROP with 'if_exists' only when it's specified by user. Otherwise, segments will abort the transaction if the table being dropped is not found, instead of just dropping table only on master.
-
由 Abhijit Subramanya 提交于
The issue was manifested by having large file in data directory. gpinitstandby encountered SIGSEGV as tar fails during basebackup. With the fix it gracefully errors out. Patch pulls in following commit from upstream in order to improve error handling and recovery in walsenders. commit fd5942c1 Author: Heikki Linnakangas <heikki.linnakangas@iki.fi> Date: Fri Oct 5 17:13:07 2012 +0300 Use the regular main processing loop also in walsenders. The regular backend's main loop handles signal handling and error recovery better than the current WAL sender command loop does. For example, if the client hangs and a SIGTERM is received before starting streaming, the walsender will now terminate immediately, rather than hang until the connection times out.
-
由 Asim Praveen 提交于
If DROP has to wait for a lock on the relation after its OID was looked up, it may happen that the name is no longer valid by the time DROP acquires the lock. This case is handled by pulling a function from upstream commit commit 4240e429 Author: Robert Haas <rhaas@postgresql.org> Date: Fri Jul 8 22:19:30 2011 -0400 Try to acquire relation locks in RangeVarGetRelid.
-
由 Ashwin Agrawal 提交于
The patch adds size required to finalize the rle repeats array size to the block space calculation for CO, to correctly detect if current block is full and should start new block for current item. If its non-repeated item (NULL / value), need to finalize the rle repeats counts array and then store new value. Space calculation only checked for space required for new item either in NULLs array or actual item space, but missed that to store the item which is non-repeating, would have to finalize the previous repeating array which would need 4 bytes of space along with it. Hence said okay to store the item but later caused failure to insert when pushing the block to disk as its over blocksize specified for the table.
-
由 Heikki Linnakangas 提交于
It was not working correctly anyway on 32-bit x86 systems, where sizeof(int) < sizeof(Datum), because in Greenplum sizeof(Datum) == 8 even on 32-bit platforms. This was revealed by failures on the 'misc' regression test, which tested this the V0 support.
-
由 Heikki Linnakangas 提交于
The 'gp_optimizer' test also creates a table called 'sales', which started to clash with the 'sales' table created by the new 'decode_expr' test. Rename the test table used in decode_expr, and also drop it at the end of the test.
-
由 Entong Shen 提交于
-
由 Heikki Linnakangas 提交于
Not sure why ORCA causes the error, but it seems to be intended behaviour, so I'm not going to try fixing that right now.
-
由 Heikki Linnakangas 提交于
The gphdfs test was concerned with testing gphdfs. It's useless without gphdfs itself, and was causing the main regression suite to fail, if gphdfs was not installed. Likewise, move the gphdfs-related parts of exttab1 test.
-
由 Heikki Linnakangas 提交于
Looks like all the 'functional' test was testing was DECODE() expressions, so rename the test, and move it to the main test suite. While we're at it, clean up the test case to some extent. Fix the missing semicolons from some queries, which caused the subsequent DROP to fail. Remove unused functions and emp and dept tables. Add some comments explaining what each test does. Remove use of multi-byte characters from test case, so that it works regardless of encoding.
-
由 Heikki Linnakangas 提交于
Remove unnecessary ORDER BY clauses that are not present in the upstream version. Disable COPY BINARY test, as GPDB doesn't support BINARY copy. Finally, adjust the error messages in the expected output to match what you get nowadays. DROP TABLE cleanup commands have been added to some other tests, so that the extra user tables don't appear in the output of the 'misc' test. It's not very nice of the 'misc' test to list all user tables, but this nevertheless seems less likely to cause merge conflicts than ripping out that part of the test altogether. There are still differences in the list of user tables though.
-
- 02 11月, 2015 7 次提交
-
-
由 Heikki Linnakangas 提交于
Revert some unnecessary changes compared to upstream: there's no need for ORDER BYs in queries, gpdiff.pl will mask out row order differences. That needed some fixes to atmsort.pl though, to make it smarter about reordering COPY TO STDOUT results. It used to only deal with the COPY (SELECT ...) TO STDOUT variant, and particularly named tables (COPY .*test1 TO STDOUT). Make it handle all COPY TO STDOUT commands. Also make it smarter about detecting the end of COPY TO STDOUT output. In addition to -- or ERROR, also treat any SELECT, UPDATE, CREATE etc. command as the end of result. In the passing, remove the "-- copy_stdout" command from atmsort.pl, as it was unused. No need to label the functions as immutable or with NO SQL or MODIFIES SQL DATA, and update the error message in the expected output about cross-segment access within functions. One of the tests gives a different error than on upstream: an empty line gives "invalid integer" error in upstream, but "missing data" on GPDB. I'm not quite sure what's causing that, but both errors seem reasonable for that case (the "missing data" is perhaps even better), so let's just memorize that difference.
-
由 Heikki Linnakangas 提交于
The upstream 'copy2' regression test exercised this, but we haven't noticed because we've disabled that test. I spotted this while trying to re-enable it. Nevertheless, let's add an explicit test for this.
-
由 Heikki Linnakangas 提交于
Integers and numerics don't need quoting in CSV-format COPY, but they should still be quoted if the FORCE QUOTE option is used. We had memorized the incorrect output as expected output of the GPDB-added parts at the end of the 'copy' regression test. Fix that, but also add an explicit test case for this.
-
由 Heikki Linnakangas 提交于
The USE_FORCE_PLAN option has been unused for a long time.
-
由 Heikki Linnakangas 提交于
The shell on my laptop produces an error like: ERROR: external table ext_stderr1 command ended with error. sh: 1: ... while the expected output is: ERROR: external table ext_stderr1 command ended with error. sh: ... (Note the missing line number, "1: "). Add a substitution pattern to the test case to mask that out. There was already a similar pattern for shells that print "line 1:".
-
由 Heikki Linnakangas 提交于
Many of the test queries were doubled: run first within an ignore-block, and then for real. That seems pointless, so remove the duplicates. Also, fix typos in comments. This hopefully fixes test failures that we've occasionally seen in the buildfarm, where the mechanism in gpdiff.pl that masks out result set ordering differences gets confused by the duplicated queries in the ignore-blocks.
-
由 Heikki Linnakangas 提交于
See earlier discussion on this on pgsql-hackers, message-id 9B946296-A2EB-4B45-A190-153F06662F8B@kineticode.com. We'd get this fix eventually as we merge with later PostgreSQL versions, but let's make life easier for people hitting this issue now. This is a backpatch of the following upstream commit, sans the MSVC build parts, as I have no environment to test that and we don't support Windows anyway. We'll get the MSVC parts later, as we merge. commit ba00ab0b Author: Andrew Dunstan <andrew@dunslane.net> Date: Sat Nov 26 15:22:32 2011 -0500 Use the preferred version of xsubpp, not necessarily the one that came with the distro version of perl. David Wheeler and Alex Hunsaker. Backpatch to 9.1 where it applies cleanly. A simple workaround is available for earlier branches, and further effort doesn't seem warranted.
-
- 31 10月, 2015 1 次提交
-
-
由 Jason Champion 提交于
-
- 30 10月, 2015 2 次提交
-
-
由 Dave Cramer 提交于
-
由 Entong Shen 提交于
Exchanging a default partition is currently not allowed because there is no validation on the data being exchanged. This commit add a GUC to enable this if the user chooses to do so.
-
- 29 10月, 2015 3 次提交
-
-
由 Heikki Linnakangas 提交于
Raymond Barbiero
-
由 Heikki Linnakangas 提交于
GP wouldn't compile with newer (definitely with 2.19) versions of glibc, due to changes introduced by commit 3a50811c2fd1aff15552c0448fff66039488fee5 [1] and more precisely [2]. open() calls were limited to 2-3 args, where master code was calling it with four. This patch removes the last argument which should be unnecesarry. [1]: http://sourceware.org/git/?p=glibc.git;a=commitdiff;h=3a50811c2fd1aff15552c0448fff66039488fee5 [2]: http://sourceware.org/git/?p=glibc.git;a=commitdiff;h=3a50811c2fd1aff15552c0448fff66039488fee5#patch6 Álvaro Hernández Tortosa
-
由 Heikki Linnakangas 提交于
In the long run, we should get rid of that dependency by rewriting the code that currently uses 'ed', but meanwhile this should be documented. Sujeet Varakhedi
-
- 28 10月, 2015 5 次提交
-
-
由 Heikki Linnakangas 提交于
PyGreSQL-4.0 is in the git repository, it should not be ignored. The other files are legacy stuff that's not included in the repository, or built as part of the build, so remove them too.
-
由 Heikki Linnakangas 提交于
Based on patch by Raymond Barbiero
-
由 Heikki Linnakangas 提交于
It's important to pay attention to compiler warnings, but -Werror isn't buying us much at the moment. Firstly, there are a lot of warnings on modern versions of gcc. I have to pass: "-Wno-error=unused-but-set-variable -Wno-error=enum-compare -Wno-error=address -Wno-error=maybe-uninitialized" to make it compile. More important than erroring out on warnings is to make sure there are no warnings to begin with. Once we get to that state, any warnings will stick out like sore thumb anyway, and you won't need -Werror to make them more annoying. We have a long way until we are 100% warning free, but most of the warnings come from upstream code. They have been fixed in later versions of PostgreSQL, so they will go away as we merge with upstream.
-
由 Jimmy Yih 提交于
-
-
- 02 12月, 2006 3 次提交
-
-
由 PostgreSQL Daemon 提交于
v8.2.0 is now released ...
-
由 Bruce Momjian 提交于
-
由 Bruce Momjian 提交于
-