Github user kaknikhil commented on a diff in the pull request:
https://github.com/apache/madlib/pull/319#discussion_r217495555
--- Diff: src/ports/greenplum/dbconnector/dbconnector.hpp ---
@@ -32,6 +32,24 @@ extern "C" {
#include "Compatibility.hpp"
+#if PG_VERSION_NUM >= 90000
+ // MADlib aligns the pointers returned by palloc() to 16-byte
boundaries
+ // (see Allocator_impl.hpp). This is done to allow Eigen vectorization
(see
+ // http://eigen.tuxfamily.org/index.php?title=FAQ#Vectorization for
more
+ // info). This vectorization has to be explicitly disabled if
pointers are
+ // not 16-byte aligned.
+ // Further, the pointer realignment invalidates a header that palloc
creates
+ // just prior to the pointer address. Greenplum after commit <> fails
due
--- End diff --
The commit id is missing from the comment.
---
