[jira] [Commented] (MADLIB-1298) MADLIB doesn't work with gp_distribution_policy changes in Greenplum 6

["Heikki Linnakangas (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MADLIB-1298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16758465#comment-16758465 ] 

Heikki Linnakangas commented on MADLIB-1298:
--------------------------------------------

This came up as a test failure in the Greenplum concourse pipeline at [https://prod.ci.gpdb.pivotal.io/teams/main/pipelines/gpdb_master/jobs/MADlib_Test_planner_centos6/builds/978.]

> MADLIB doesn't work with gp_distribution_policy changes in Greenplum 6
> ----------------------------------------------------------------------
>
>                 Key: MADLIB-1298
>                 URL: https://issues.apache.org/jira/browse/MADLIB-1298
>             Project: Apache MADlib
>          Issue Type: Bug
>            Reporter: Heikki Linnakangas
>            Priority: Major
>
> Recent commits in Greenplum master branch - which will eventually become Greenplum 6 - changed the gp_distribution_policy catalog table's layout. See [https://github.com/greenplum-db/gpdb/commit/69ec6926c2284ff97d8b82282072b5b205e8aa27] and [https://github.com/greenplum-db/gpdb/commit/242783ae9f573bb03da3d89dc689c6743dfce510.]
>  
> That broke get_distribution_policy() function, in utilities.py_in, because it looks at the renamed gp_distribution_policy.attrnums column. The recommended way to deparse the DISTRIBUTED BY clause of a table is now to use the new pg_catalog.pg_get_table_distributedby() helper function. MADLIB should use that too, when running against GPDB 6.
>  
> (I'll open a pull request for this shortly)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)