This reverts commit cd2cfa7d.

This reverts commit 83a2f870.

This reverts commit 3aa7de6a.

-- including all text references
Signed-off-by: NMelanie Plageman <mplageman@pivotal.io>

These are nightly triggers.

Full Duration: 1.5 hours
Incremental Duration: 3.5 hours
MFR Duration: 40 mins

Renames 4.3 sql and answer files since these tests are now versioned
with the code and deletes old 4.2 files.

Signed-off-by: NMarbin Tan <mtan@pivotal.io>

The file postgresql.conf contains GUC settings which are applied upon GPDB restart.
--file-compare contrasts the current GPDB settings (stored in the database)
to settings in the file which would be applied upon restart.

For almost all GUCs, all segments should have the same value; the --file-compare flag
either summarizes success (when synchronized) or, in the case of
unexpected differences, outputs detailed information about a given GUC
on all segments.
Signed-off-by: NMarbin Tan <mtan@pivotal.io>

Signed-off-by: NMarbin Tan <mtan@pivotal.io>

On our normal workstations, gpsmon comes up within 5 seconds very
consistently, but in other environments, it might need more time
Signed-off-by: NC.J. Jameson <cjameson@pivotal.io>

The xpath test in general is fairly questionable:
  - it largely overlaps in coverage with the xml test from upstream
  - it contains a few negative assertions that depend on an error
    message that doesn't even come from our (Postgres) code: it's an
    implementation detail of libxml2. On CentOS, a few Red Hat patches
    to libxml2 always make those tests fail.

This commit removes the xpath test.

[ci skip]

`optimizer_explain_show_status` GUC is guarding the optimizer
status string in the explain output. There is no added value
for guarding the optimizer status output with a GUC. Therefore,
we are removing it.
Signed-off-by: NOmer Arap <oarap@pivotal.io>

1. fix an error if the quote and escape options are not specified in the
CSV format.

2. add codes for delimiter, escape and quote options to process a
ASCII, unicode-encoded or escaped single character in order to find
the correct table to reuse when load external data.

1. Support num_running, num_queueing, num_queued, num_executed and
total_queue_duration in gp_toolkit.gp_resgroup_status.
2. Reflect status in pg_stat_activity when transaction is queued in resource group.
Signed-off-by: NRichard Guo <riguo@pivotal.io>
Signed-off-by: NKenan Yao <kyao@pivotal.io>

gpddboost will fail to upload an empty (0 byte) file to Data Domain.
Now, in cases where we previously uploaded an empty file we insert a
newline to allow gpddboost to upload successfully. We also update the
logic on incremental backups to correctly parse these files.

Authors: Chris Hajas and Karen Huddleston

- Use independent ext and sigar from artifactory
- zlib was not used anymore
[#143414851]
[#143441561]
Signed-off-by: NTom Meyer <tmeyer@pivotal.io>

Pivnet requires sha256 checksums for uploaded objects.

[#143368023] https://www.pivotaltracker.com/story/show/143368023Signed-off-by: NTom Meyer <tmeyer@pivotal.io>

These were missed from commit 8e02d076.
Also removes backup files from /tmp directory.

Authors: Karen Huddleston and Chris Hajas

[ci skip]

* Add psql external table meta-command \dE for external tables and change \dx to extensions.

* Changes an additional 'x' to 'E'.

These were creating complex DDL that is covered by other tests. This shortens the suite by ~5 minutes.

This should be removed when resource group feature is completed later.

* If a user builds dependencies into an arbitrary directory, allow
  CFLAGS to propagate that information to the compiler and linker.
Signed-off-by: NMarbin Tan <mtan@pivotal.io>

With cd2cfa7d (disabling heterogeneous index scans) , now we can make
stronger assumption about the usefulness of partial logical indices and
return only complete logical indices for a partitioned table.

Now that we don't use optimizer_plan_id, also stop setting
optimizer_enable_space_pruning and optimizer_enumerate_plans. This
should speed up our test quite a bit.

(cherry picked from commit 74b90db3281c6e7fb3240a4ad565d3ebbd46b0a5)

Another bug in atmsort will take three dashes as an indicator of the
tabular separator for plan output, and that will suppress the normal
processing of `start_ignore` and other directives. This produces very
very useless test failures.
This commit fixes that.

atmsort.pl kinda indiscriminately treat lines (even comments) containing
the character sequence EXPLAIN...SELECT as a hint of the following lines
being a plan. Which is wrong, when the word selector also matches that.

Changing the comments for now, before we really fix atmsort. This will
give us much better failures.