GitHub user iyerr3 opened a pull request:
https://github.com/apache/madlib/pull/340
Validation: Support other 'relkind' for input tables
JIRA: MADLIB-1287
Within `validate_args.py_in:table_exists()` we checked if a table existed
within
`pg_class` but limited the input table to specific `relkind`s. This limited
scope is unnecessary and precluded MADlib functions from accepting partition
tables.
This commit removes the `relkind` check, effectively adding 'partition',
'index' and 'sequence' tables as valid input tables.
Closes #340
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/iyerr3/madlib bugfix/support_partition_tables
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/madlib/pull/340.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #340
----
commit 88d53fdf22a5e79120580c171279f077b36b267d
Author: Rahul Iyer <riyer@...>
Date: 2018-12-11T13:24:17Z
Validation: Support other 'relkind' for input tables
JIRA: MADLIB-1287
Within `validate_args.py_in:table_exists()` we checked if a table existed
within
`pg_class` but limited the input table to specific `relkind`s. This limited
scope is unnecessary and precluded MADlib functions from accepting partition
tables.
This commit removes the `relkind` check, effectively adding 'partition',
'index' and 'sequence' tables as valid input tables.
Closes #340
----
---
