1. 21 3月, 2017 4 次提交
  2. 20 3月, 2017 5 次提交
  3. 18 3月, 2017 4 次提交
    • K
      Add assert that number of sort columns is 1 always for range in window · b533bcb2
      Karthikeyan Jambu Rajaraman 提交于
      functions.
      
      Coverity complains that the function `cmp_deformed_tuple` access the
      datum from expression eval as an array. But we don't have that use case due to the
      following error. For e.g., the parser will complain that it can't have
      more than 1 columns for sorting when we use range in window function as shown below.
      
      ```
      select depname, empno, salary, LAST_VALUE(salary)
      error.
      over (partition by depname order by empno, salary range
      between 5000 preceding and 3900 preceding) as
      highest_salary from empsalary;
      ERROR:  only one ORDER BY column may be specified when RANGE is used in
      a window specification
      ```
      b533bcb2
    • T
      Update paths for installers to avoid regexp match collisions · a848f1b3
      Tushar Dadlani 提交于
      - In the current system, greenplum-db-appliance-(.*) collides with
        greenplum-db-(.*)
      Signed-off-by: NTom Meyer <tmeyer@pivotal.io>
      a848f1b3
    • D
      Multi-dimensional arrays can now be used as arguments to a PL/python … (#1292) · eb1740c6
      Dave Cramer 提交于
      * Multi-dimensional arrays can now be used as arguments to a PL/python function
      (used to throw an error), and they can be returned as nested Python lists.
      
      This makes a backwards-incompatible change to the handling of composite
      types in arrays. Previously, you could return an array of composite types
      as "[[col1, col2], [col1, col2]]", but now that is interpreted as a two-
      dimensional array. Composite types in arrays must now be returned as
      Python tuples, not lists, to resolve the ambiguity. I.e. "[(col1, col2),
      (col1, col2)]".
      
      To avoid breaking backwards-compatibility, when not necessary, () is still
      accepted for arrays at the top-level, but it is always treated as a
      single-dimensional array. Likewise, [] is still accepted for composite types,
      when they are not in an array. Update the documentation to recommend using []
      for arrays, and () for composite types, with a mention that those other things
      are also accepted in some contexts.
      
      The upstream patch is 94aceed3
      
      * Give a hint, when [] is incorrectly used for a composite type in array.
      
          That used to be accepted, so let's try to give a hint to users on why
          their PL/python functions no longer work.
      
         see upstream 510e1b8e
      
      * add tests for additional conversions.
      move them into plpython_types
      move the create function closer to the actual select
      which is more like upstream
      eb1740c6
    • D
      Use available Makefile variables for src/bin · 34788e9a
      Daniel Gustafsson 提交于
      Since we have a boolean yes/no switch for coverage enabled builds
      in Makefile.global from autoconf, use that rather than inspecting
      the CFLAGS for ease of reading the code. Also use the top_builddir
      variable in setting the libdir, while unlikely to move in the
      hierarchy it's the right thing to do.
      34788e9a
  4. 17 3月, 2017 17 次提交
  5. 16 3月, 2017 5 次提交
    • P
      auto-completion for CREATE/DROP RESOURCE GROUP. · 0c62d8b0
      Pengzhou Tang 提交于
      Signed-off-by: NNing Yu <nyu@pivotal.io>
      0c62d8b0
    • N
      Implement DROP RESOURCE GROUP syntax. · 01565b0a
      Ning Yu 提交于
      Any resgroups created by CREATE RESOURCE GROUP syntax can be dropped
      with DROP RESOURCE GROUP syntax; the default resgroups, default_group
      and admin_group, can't be dropped; only superuser can drop resgroups;
      resgroups with roles bound to can't be dropped.
      
          -- drop a resource group
          DROP RESOURCE GROUP rg1;
      
      *NOTE*: this commit only implement the DROP RESOURCE GROUP syntax, the
      actual resource management are not yet supported, which will be provided
      later based on these syntax commits.
      
      *NOTE*: test cases are provided for both CREATE and DROP syntax.
      Signed-off-by: NPengzhou Tang <ptang@pivotal.io>
      01565b0a
    • P
      Implement CREATE RESOURCE GROUP syntax. · 8ab1ccdb
      Pengzhou Tang 提交于
      There are two default resource groups 'default_group' and 'admin_group',
      to create more please use the CREATE RESOURCE GROUP command, group
      options can be specified with the WITH clause, at the moment
      'cpu_rate_limit' and 'memory_limit' are mandatory, other options are
      all optional.
      
          -- create a resource group
          CREATE RESOURCE GROUP rg1 WITH (
              concurrency=1,
              cpu_rate_limit=.2,
              memory_limit=.2
          );
          -- query the resource group
          SELECT oid FROM pg_resgroup WHERE rsgname='rg1';
          SELECT * from gp_toolkit.gp_resgroup_config
              WHERE groupname='rg1';
          -- create/alter a role and assign it to this group
          CREATE ROLE r1 RESOURCE GROUP rg1;
          ALTER ROLE r2 RESOURCE GROUP rg1;
      
      *NOTE*: this commit only implement the SQL syntax, the actual resource
      limitation will not take effect at the moment as the resource group is
      still under development.
      
      *NOTE*: test cases are not included in this commit as once a testing
      resgroup is created it can't be dropped due to lack of DROP syntax, so
      the test case can't be re-run and will introduce side-effect to the
      system. So it's better to provide test cases after the DROP RESOURCE
      GROUP is implemented.
      Signed-off-by: NNing Yu <nyu@pivotal.io>
      8ab1ccdb
    • A
      Restrict the gpfdist sslclean workaround to cmdline only · 78c29ff4
      Adam Lee 提交于
      gpfdist waits 5 seconds to close SSL sessions to workaround a system
      related issue on Solaris, this commit restricts the workaround to
      cmdline only.
      Signed-off-by: NAdam Lee <ali@pivotal.io>
      Signed-off-by: NYuan Zhao <yuzhao@pivotal.io>
      78c29ff4
    • T
      Make git submodules clonable recursively without github keys · eb888c69
      Tushar Dadlani 提交于
      - If you are working on a computer and want to get the entirety of the
        source code of gpdb including submodules, you cannot do that as you would need your github
        key to clone the submodules.
      eb888c69
  6. 15 3月, 2017 5 次提交
    • D
      rebased to upstream tag REL8_4_22 (#2011) · 614ca952
      Dave Cramer 提交于
      fix tests, add init_file to ignore GPDB segment output
      use init_file from main regression tests, fix expected results
      Requires GPDB to be configure with --with-openssl to pass
      614ca952
    • D
      Fix spelling of commandline option in gpfdist · bcac963c
      Daniel Gustafsson 提交于
      bcac963c
    • H
      Use the new optimizer_trace_fallback GUC to detect ORCA fallbacks. · bb3e4b4f
      Heikki Linnakangas 提交于
      qp_olap_group2 did an elaborate dance with optimizer_log=on,
      client_min_messages=on, and gpdiff rules to detect whether the queries
      fell back to the traditional planner. Replace all that with the new simple
      optimizer_trace_fallback GUC.
      
      Also enable optimizer_trace_fallback in the 'gp_optimizer' test. Since this
      test is all about testing ORCA, seems appropriate to memorize which queries
      currently fall back and which do not, so that we detect regressions where
      we start to fall back on queries that ORCA used to be able to plan. There
      was one existing test that explicitly set client_min_messages=on, like the
      tests in qp_olap_group2, to detect fallback. I kept the those extra logging
      GUCs for that one case, so that we have some coverage for that too, although
      I'm not sure how worthwhile it is anymore.
      
      In the passing, in the one remaining test in gp_optimizer that sets
      client_min_messages='log', don't assume that log_statement is set to 'all'.
      
      Setting optimizer_trace_fallback=on for 'gp_optimizer' caught the issue
      I fixed in previous commit, that one of the ANALYZE queries still used ORCA.
      bb3e4b4f
    • H
      Disable ORCA also for the internal query to get index size. · 702d5683
      Heikki Linnakangas 提交于
      For the same reasons we disabled ORCA in all the other ANALYZE queries.
      Missed this one.
      
      I belive we don't want to use ORCA for any of the ANALYZE's internal
      queries, current or future ones, so move the disabling of ORCA one level
      up, into a wrapper around analyze_rel().
      702d5683
    • E
      Fix error in pgbouncer doc example · 2be49a69
      Eamon 提交于
      [ci skip]
      2be49a69