- 01 9月, 2017 31 次提交
-
-
由 Daniel Gustafsson 提交于
This reverts commit 1596d323 since it broke cluster startup, clearly it needs more attention before being ready.
-
由 Daniel Gustafsson 提交于
This puts on on par with upstream PostgreSQL 9.1.20, released on 2016-02-11, for tzdata and the below referenced commit. The reason for not updating further is that there are tzdata format changes in 9.1.21, 2016a is the last release we can import without merging code changes as well. commit 6887d72d06a2f36508d5be9cca316088d4c60b26 Author: Tom Lane <tgl@sss.pgh.pa.us> Date: Fri Feb 5 10:59:09 2016 -0500 Update time zone data files to tzdata release 2016a. DST law changes in Cayman Islands, Metlakatla, Trans-Baikal Territory (Zabaykalsky Krai). Historical corrections for Pakistan.
-
由 Daniel Gustafsson 提交于
This bumps the copyright years to the appropriate years after not having been updated for some time. Also reformats existing code headers to match the upstream style to ensure consistency.
-
由 Daniel Gustafsson 提交于
The linenumbers are prone to change (and ideally shouldn't be there at all since we should use ereport instead), so scrub them from the test output with a matchsubs rule. The recent changes for copyright notices broke this since it added a line to the file.
-
由 Daniel Gustafsson 提交于
The missing errcode makes the ereport call include the line number of the invocation from the .c file, which not only isn't too useful but cause the tests to fail on adding/removing code from the file.
-
由 Peifeng Qiu 提交于
The original gpload test suites is intended to be run locally, so gpdb cluster and test must be on the same machine. For platform that doesn't have GPDB server available but have loader package, we need to run the test remotely.
-
由 Heikki Linnakangas 提交于
* Use ereport() with a proper error code, rather than elog(), so that you don't get the source file name and line number in the message, and the serious-looking backtrace in the log. * Remove the hint that advised "SET gp_enable_segment_copy_checking=off", when a row failed the check that it's being loaded to the correct segment. Ignoring the mismatch seems like very bad idea, because if your rows are in incorrect segments, all bets are off, and you'll likely get incorrect query results when you try to query the table.
-
由 Pengzhou Tang 提交于
-
由 Pengzhou Tang 提交于
When define index concurrently, DefineIndex() commits current transaction and dispatch a command to segments without resource group info, therefore, QE get a capability info with zero configured, the problem is groupReleaseMemQuota() use the empty capability info wrong and caused a floating-point exception. According to the code, it should use prevSharedInfo->caps instead
-
由 Lav Jain 提交于
* Make PXF part of the gpdb binary * Fix popd error
-
由 Mel Kiyama 提交于
* DOCS: SELECT - Add RECURSIVE keyword to WITH clause * docs: recursive CTE - add experimental designation. Add GUC gp_recursive_cte_prototype * docs: updated recursive CTE docs based on review comments.
-
由 Marbin Tan 提交于
Signed-off-by: NNadeem Ghani <nghani@pivotal.io>
-
由 John Gaskin 提交于
-
由 Daniel Gustafsson 提交于
-1 is a valid value for errordata_stack_depth until errstart() has been called which inits the stack. In correct usage it's thus fine to use it as an array subscript without inspecting it, since demote and dismiss are useless outside an error context. Incorrect usage would however cause a memory error so guard against that since it's the right thing to do.
-
由 Daniel Gustafsson 提交于
The memory allocated is immediately orhpaned since the pointer is overwritten. Remove and avoid leaking.
-
由 Daniel Gustafsson 提交于
strncpy() can render the buffer non-NULL terminated on input which is MAXPGPATH characters long. This is not only incorrect code, it's a mismerge since upstream has never used strncpy here, but strlcpy. Fix by replacing with strlcpy calls, thus aligning us with upstream better as well.
-
由 Daniel Gustafsson 提交于
The num_cwords member was left blank which cause issues for the freeing function which operates based on its value.
-
由 Daniel Gustafsson 提交于
{f}stat() can fail and reading the stat buffer without checking for status is bad hygiene. Ensure to always test return value and take the appropriate error path in case of stat error.
-
由 Daniel Gustafsson 提交于
Iff the value isn't a varlena, then dataLen and dataStart won't be set but we may still come to inspect them later on. Avoid reading uninitialized variables by zeroing them in that case and check for that before reading.
-
由 Daniel Gustafsson 提交于
The strncpy() will make path2 non-NULL terminated in the (rare?) cases where filespaceLocation2 is MAXPGPATH chars long. Use strlcpy instead like other places in the codepath does and ensure we always have a NULL terminated string.
-
由 Larry Hamel 提交于
As part of a previous commit, the WorkerPool of threads will raise if the number of workers is set to 0. This prevents a coding error from resulting in no work, where work was expected. This commit reaches through code to protect against the numWorkers parameter being 0. In several cases, the number of workers is set to the number of segments or the number of databases. We do not protect those with the expectation that something is more significantly wrong if those sums are 0, and that an exception would be fitting in those cases. Signed-off-by: NMarbin Tan <mtan@pivotal.io>
-
由 Marbin Tan 提交于
Signed-off-by: NNadeem Ghani <nghani@pivotal.io>
-
由 mkiyama 提交于
-
由 Larry Hamel 提交于
Rework how environment variables transmitted in local bash and remote ssh commands. If you have a bash command that starts with a conditional, you'll get a syntax error. To fix this, we added ampersands to join the setting of the environment variables and the command itself. NOTE: We removed an ExecutionContext static variable that recorded environment variables. This feature was not used anywhere. Signed-off-by: NMarbin Tan <mtan@pivotal.io>
-
由 Shoaib Lari 提交于
Previously, RemoveDirectories and RemoveFiles used the unix command "rm -rf", but this is inefficient for huge numbers of files. Also, these functions accepted any globbed path. Instead, use "rsync" to optimize deletion of files in a directory. On a DCA using 1 million files, this increased speed by about 3x. Also, this commit breaks up the different use-cases of deletion into separate methods, adding methods RemoveDirectoryContents() and RemoveFile() and RemoveGlob() to help isolate the assumptions of each case and optimize for them. Signed-off-by: NLarry Hamel <lhamel@pivotal.io> Signed-off-by: NShoaib Lari <slari@pivotal.io>
-
由 Larry Hamel 提交于
This utility is slated for removal in v6. It is unused in general. Fix obvious compile errors and basic logic for logging. Signed-off-by: NShoaib Lari <slari@pivotal.io>
-
由 Shoaib Lari 提交于
Signed-off-by: NLarry Hamel <lhamel@pivotal.io>
-
由 Marbin Tan 提交于
Signed-off-by: NLarry Hamel <lhamel@pivotal.io>
-
由 Larry Hamel 提交于
Signed-off-by: NMarbin Tan <mtan@pivotal.io> Signed-off-by: NLarry Hamel <lhamel@pivotal.io>
-
由 Jimmy Yih 提交于
Fix Travis by adding --disable-pxf to configure command. We could always just add json-c-devel package and see if it works... but we need to determine if we want Travis to compile pxf or not first. We currently have Travis set up to not compile gpAux/extensions so this PR just adds to the current setup.
-
由 Chuck Litzell 提交于
* docs: Add checksums overview to Admin Guide * Add sentence and xref about gpstart verifying checksums. * Fixing broken href to gpstart.xml * Edits from review comments
-
- 31 8月, 2017 9 次提交
-
-
由 Lav Jain 提交于
-
由 Jimmy Yih 提交于
Current FTS probe relies on file replication data structures mostly found in pmModuleState. This commit establishes an empty FTS probe response packet that primaries using segment WAL replication can fill out later. We also introduce the 'w' fault_strategy (WAL replication) to replace the 'f' fault_strategy (file replication). This commit is carved and modified from a previously closed PR: https://github.com/greenplum-db/gpdb/pull/2936 Authors: Abhijit Subramanya and Jimmy Yih
-
由 David Yozie 提交于
-
由 Lav Jain 提交于
* Refactor GPHDFS regression to run for pxf * remove customized Hadoop home location * PXF tarball creation inside GPDB pipeline * Remove legacy directory * Use enable_pxf instead of with_pxf
-
由 Daniel Gustafsson 提交于
-
由 Bhuvnesh 提交于
* Change the directory location and update readme When conan is used to build the dependencies (orca & xerces) of gpdb, it copies the headers and libraries to the path specified in the imports section of the conanfile.txt. Changing the target copy location to /usr/local/include and /usr/local/lib as its the default for gpdb. In case the user prefers to have a different directory, they can change the location accordingly.
-
由 Heikki Linnakangas 提交于
Rather than appending to a StringInfo, return a string. The caller can append that to a StringInfo if he wants to. And instead of passing a prefix as argument, the caller can prepend that too. Both callers passed the same format string, so just embed that in the function itself. Don't append a trailing "; ". It's easier for the caller to append it, if it's preferred, than to remove it afterwards. Also add a regression test for the 'gp_enable_fallback_plan' GUC. There were none before. The error message you get with that GUC disabled uses the gp_guc_list_show function.
-
由 Heikki Linnakangas 提交于
-
由 Heikki Linnakangas 提交于
Plus other minor cleanup.
-