Permalink
Browse files

Convex: Remove remove LMF merge temporarily

Jira: MADLIB-756

The current implementation fares poorly on Greenplum instances
with many segments. The problem has been identified with the merge
function, and is temporarily disabled till a fix is found.
  • Loading branch information...
1 parent f6f9517 commit e6125f8874c38597d870f9dc0d4a7d0ea40d90d7 Rahul Iyer committed Nov 14, 2012
Showing with 3 additions and 1 deletion.
  1. +2 −0 src/ports/greenplum/4.0/config/Modules.yml
  2. +1 −1 src/ports/postgres/modules/convex/lmf.sql_in
@@ -25,6 +25,8 @@ modules:
depends: ['utilities']
- name: sample
depends: ['utilities']
+ - name: convex
+ depends: ['utilities']
- name: sketch
- name: stats
- name: svd_mf
@@ -191,7 +191,7 @@ CREATE AGGREGATE MADLIB_SCHEMA.lmf_igd_step(
/*+ scale_factor */ DOUBLE PRECISION) (
STYPE=DOUBLE PRECISION[],
SFUNC=MADLIB_SCHEMA.lmf_igd_transition,
- m4_ifdef(`__GREENPLUM__',`prefunc=MADLIB_SCHEMA.lmf_igd_merge,')
+ -- m4_ifdef(`__GREENPLUM__',`PREFUNC=MADLIB_SCHEMA.lmf_igd_merge,')
FINALFUNC=MADLIB_SCHEMA.lmf_igd_final,
INITCOND='{0,0,0,0,0,0,0,0,0}'
);

0 comments on commit e6125f8

Please sign in to comment.