- 13 5月, 2019 4 次提交
-
-
由 Adam Berlin 提交于
-
由 Hubert Zhang 提交于
When determine waiting relation in global dead lock detector, we should also check whether the waiter's lockmode conflict with holder's holdmask. Co-authored-by: NNing Yu <nyu@pivotal.io> Reviewed-by: NZhenghua Lyu <zlv@pivotal.io>
-
由 Bhuvnesh Chaudhary 提交于
-
由 Bhuvnesh Chaudhary 提交于
-
- 12 5月, 2019 2 次提交
-
-
由 Daniel Gustafsson 提交于
The utilities reference page contained (unlinked) mentions of long since deprecated utilities, and since they aren't shipping it's time to remove them: gpsizecalc - removed in June 2010 gpskew - removed in October 2009 gprebuildsystem - removed in January 2010 gpchecknet - removed in April 2010 gpcheckos - removed in July 2016 Reviewed-by: Lisa Owen Reviewed-by: David Yozie
-
由 Daniel Gustafsson 提交于
gpdetective was removed in 4.3.5, so it seems about time to also remove the documentation for it (which was unreachable due to the app being marked deprecated). Reviewed-by: Lisa Owen Reviewed-by: David Yozie
-
- 11 5月, 2019 2 次提交
-
-
由 Venkatesh Raghavan 提交于
-
由 Taylor Vesely 提交于
There was a race condition where replay of a DROP TABLESPACE command on a replication mirror could occur after the tests had already physically removed the directories. It should be safe if the tablespace directories continue to exist after the test is complete, so don't clean them up.
-
- 10 5月, 2019 12 次提交
-
-
由 Lisa Owen 提交于
* docs - add pageinspect module landing page * remove peer scope * cannot inspect ao or external relations
-
由 Daniel Gustafsson 提交于
Commit 0b6f15b8 removed the only callsite of memtuple_get_nullp_len(), so remove the function too to avoid a compiler warning (and reduce cruft). Reviewed-by: NTang Pengzhou <ptang@pivotal.io> Reviewed-by: NAshwin Agrawal <aagrawal@pivotal.io>
-
由 Daniel Gustafsson 提交于
This moves the errorpath for EXTERNAL TABLE in LIKE INCLUDING in under the context aware error handling. This makes the error when creating an external table the same as when creating a foreign table. As a bonus it avoids a few checks in case of error. Reviewed-by: Jimmy Yih Reviewed-by: Asim R P
-
由 Daniel Gustafsson 提交于
INCLUDING STORAGE was introduced in PostgreSQL 9.0, but support for setting the AO/AOCS/ENCODING storage options on the created table was never supported as we merged upstream. This extends support for copying over the table storage type as well as attribute encodings when creating a table with INCLUDING STORAGE or ALL. Reported-by: Cyrille Lintz Reviewed-by: Jimmy Yih Reviewed-by: Asim R P
-
由 Daniel Gustafsson 提交于
-
由 Daniel Gustafsson 提交于
The pageinspect code only knows how to handle heap relations, so make sure to error out gracefully on AO/AOCS and EXTERNAL relations. Reported-by: NLisa Owen <lowen@pivotal.io> Reviewed-by: NAsim R P <apraveen@pivotal.io>
-
由 Daniel Gustafsson 提交于
s/specifed/specified/
-
由 Weinan WANG 提交于
At present, `gpstop` check `pg_stat_activity` table before stop cluster in smart model. However, we need to ignore those background workers that are part of the database implementation itself. For this purpose, we maintain a whitelist using its `application_name` field.
-
由 Lisa Owen 提交于
-
由 Jesse Zhang 提交于
The "combine" function for the int4 sum/avg aggregate functions is backported to Greenplum 6.0 in commit 313cef6e (from postgres/postgres@11c8669c0cc) but we made the inevitable omission of setting the "strictness" of int4_avg_combine to false. This by itself is harmless as the actual function body *does* guard against NULL input, but it also prohibits a whole host of optimizations when the executor and planner can detect NULL input early on and short-circuit the execution. Oops. This patch flips the `proisstrict` flag back to true for int4_avg_combine. Backpatch to 6X_STABLE.
-
由 Sambitesh Dash 提交于
optimizer_enable_dml is set to true by default. When set to false, ORCA will fall back to planner for all DML queries.
-
由 Lisa Owen 提交于
* docs - add to pxf upgrade procedure (v5.0.1 to v5.3.2) * remove extraneous to
-
- 09 5月, 2019 11 次提交
-
-
由 Daniel Gustafsson 提交于
When referring to the product name and not a database instance running Greenplum, the capitalization should be "Greenplum Database".
-
由 Asim R P 提交于
-
由 Asim R P 提交于
The --load-extension option was misspelt as load_extension. Resource group tests are difficult to run locally, neither are they run in PR pipeline. That makes it difficult to catch such errors.
-
由 Asim R P 提交于
This test has failed at least once due to the terminate query being executed before the to be terminated 'create table' statement. This was evident from master logs. The commit makes it more reliable by injecting a fault and waiting for the fault to be trigggered before executing pg_terminate_backend(). As a side benefit, we no longer need to create any additional table.
-
由 Asim R P 提交于
The extension is created by the test harness, after creating the regression / isolation2test databases. Tests should directly start using the extension and not attempt to create it. In addition to simplifying tests a little bit, this change avoids an error (duplicate key value violates unique constraint on pg_extension) when two or more tests execute create extension command concurrently. The error is not a problem in practice, it expected because of the way create extension DDL works. It is, however, unacceptable for regress-style tests that expect a deterministic response to each SQL command.
-
由 Sambitesh Dash 提交于
-
由 Pengzhou Tang 提交于
In pg_get_expr(), after getting the relname, if the table that the relid tells is dropped, an error will raise later when opening the relation to get column names. pg_get_expr() is used by GPDB add-on view 'pg_partitions' which is widely used by regression tests for partition tables. Lots of parallel test cases issue view pg_partitions and drop partition tables concurrently, so those cases are very flaky. Serialize test cases will cost more testing time and be fragile, so GPDB holds a AccessShareLock here to make tests stable.
-
由 Pengzhou Tang 提交于
Assume user1 has the privilege to database db1 and user2 has not, when user1 try to create a schema in db1 and authorize it to user2, a permission denied error is reported in QE. The RCA is QD set current user to user2 before dispatching the query to QEs, so QE will also set current user to user2, however, user2 has no provilege to create schema in database db1. To fix this, we delay setting the current user to user2 until the query is dispatched to QEs.
-
由 Pengzhou Tang 提交于
GPDB used to allow command like "START_REPLICATION %X/%X [SYNC]" to start a replication, user can specify SYNC option to skip waiting synchronous in replications. Now start replication command is made similar to upstream, the SYNC option is not supported, however, the internal flag "synchronous" is still used and always be false which make master and standby never synchronized.
-
由 Karen Huddleston 提交于
Some of the task files were used by jobs in the orca pipeline, but those jobs have been removed so the files are not being used anymore. Co-authored-by: NKaren Huddleston <khuddleston@pivotal.io> Co-authored-by: NDavid Sharp <dsharp@pivotal.io>
-
由 David Yozie 提交于
-
- 08 5月, 2019 9 次提交
-
-
由 Daniel Gustafsson 提交于
Ensure that error messages in src/backend/parser follow the upstream guidelines for formatting, styling and content. * Start hints and details with uppercase and end with period * Start messages with lowercase with no ending period * Avoid breaking messages in code to make grepping easier This also cleans up the worst whitespace offences around error messages. Reviewed-by: NTang Pengzhou <ptang@pivotal.io>
-
由 Daniel Gustafsson 提交于
-
由 Daniel Gustafsson 提交于
Initializing a struct with zeroes using {0} is legal in C99, but cause a warning in Clang when the first member of the struct isn't a scalar variable: gpfdist.c:1299:44: warning: suggest braces around initialization of subobject [-Wmissing-braces] struct fstream_filename_and_offset fos = {0}; ^ {} This is a bug in Clang, but we also don't want to turn off this class of warnings due to a this trivial false positive, as that might hide actual warnings somewhere. Since the struct in question is guaranteed to be set before being read, we can avoid the explicit initialization.
-
由 Daniel Gustafsson 提交于
The value set in default_pos was never used, so remove the variable entirely. The consumers were most likely removed in a refactoring at some point. Reviewed-by: NJimmy Yih <jyih@pivotal.io>
-
由 Daniel Gustafsson 提交于
Allocating memory manually with malloc() requires checking that the allocation was granted before dereferencing the pointer. To fix use pg_malloc() which is guaranteed to return a valid pointer or error out. Also update the reclaim to use a matching pg_free() call. Reviewed-by: NJimmy Yih <jyih@pivotal.io>
-
由 Peifeng Qiu 提交于
- Copy and package dependency DLLs - Add gpload.bat to make gpload runnable in cmd - Fix pygresql build script - Replace gpload version variable
-
由 Zhenghua Lyu 提交于
Lockmode of Update|Delete or Select-for-update Statement is controlled by: whether the table is AO or heap and the GUC gp_enable_global_deadlock_detector. The logic for lockmode is: 1. Select-for-update always hold ExclusiveLock 2. UPDATE|DELETE on AO tables always hold ExclusiveLock 3. UPDATE|DELETE on heap tables hold ExclusiveLock when gp_enable_global_deadlock_detector is off, otherwise hold RowExclusiveLock We hold locks in parser stage and Initplan before executing, the lockmode should be the same at the two stages. This commit fixes lockmode issues to make things correct. Co-authored-by: NShujie Zhang <shzhang@pivotal.io>
-
由 David Yozie 提交于
* Docs: fixing several broken links * Docs: fixing several broken links * Adding info about GPORCA support notice * add missing comma
-
由 Taylor Vesely 提交于
Force the mirror to create a restartpoint, and as a side-effect replay the DROP TABLESPACE DDL before removing the tablespace directory. Co-authored-by: NSoumyadeep Chakraborty <sochakraborty@pivotal.io>
-