Re: [VIDEO REPLAY] MADlib Call - Ask Us Anything (About MADlib)

2016-07-27 Thread Gregory Chase
Today's open office hours call turned into a delightful discussion with a new user to MADlib. Other new users to MADlib may appreciate his questions. Here's the full replay: https://youtu.be/EPQk5Uzs8OY?t=35s Here's the topics we talked about: 0:35 Our new users’ use case

[GitHub] incubator-madlib issue #56: Path: Add support for overlapping patterns

2016-07-27 Thread njayaram2
Github user njayaram2 commented on the issue: https://github.com/apache/incubator-madlib/pull/56 LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or i

[GitHub] incubator-madlib issue #56: Path: Add support for overlapping patterns

2016-07-27 Thread mktal
Github user mktal commented on the issue: https://github.com/apache/incubator-madlib/pull/56 @njayaram2 I removed default values in python and changed default values of boolean from `NULL` to `False` in sql file. Now in python we can just write `not partition_expr` to check if it is `

[GitHub] incubator-madlib pull request #59: Path: Fixing online help

2016-07-27 Thread njayaram2
GitHub user njayaram2 opened a pull request: https://github.com/apache/incubator-madlib/pull/59 Path: Fixing online help Adding examples to the path function's online help. @iyerr3 @fmcquillan99 @mktal Can you please merge this with other changes you are making in path?

[GitHub] incubator-madlib pull request #58: Sessionize function: Handling NULL for pa...

2016-07-27 Thread njayaram2
GitHub user njayaram2 opened a pull request: https://github.com/apache/incubator-madlib/pull/58 Sessionize function: Handling NULL for partition_expr JIRA: MADLIB-1001 Changes here now set partition_expr to partition across the whole table if it is originally set to NULL

[GitHub] incubator-madlib pull request #56: Path: Add support for overlapping pattern...

2016-07-27 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/56#discussion_r72513854 --- Diff: src/ports/postgres/modules/utilities/path.py_in --- @@ -40,7 +40,7 @@ m4_changequote(`') def path(schema_madlib, source_table,

[GitHub] incubator-madlib pull request #56: Path: Add support for overlapping pattern...

2016-07-27 Thread mktal
Github user mktal commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/56#discussion_r72507222 --- Diff: src/ports/postgres/modules/utilities/path.py_in --- @@ -40,7 +40,7 @@ m4_changequote(`') def path(schema_madlib, source_table, outp

Re: [VIRTUAL] MADlib Call TODAY 7/27 9AM PT: Ask Us Anything (About MADlib)

2016-07-27 Thread Greg Chase
Greetings MADlib community, This is a reminder that our next community virtual meeting starts in about an hour. Topic: "Ask Us Anything (About MADlib)". Bring your questions. You can join live here -Greg On Wed, Jul 20, 2016 at 11:18 AM, Greg

[GitHub] incubator-madlib issue #57: MADLIB-944: Add PostgreSQL 9.5 and 9.6 support

2016-07-27 Thread snaga
Github user snaga commented on the issue: https://github.com/apache/incubator-madlib/pull/57 @iyerr3 I think the code of madlib.array_agg() function is shared among PostgreSQL and GPDB. PostgreSQL (9.3~9.6) can work without this function, but GPDB still needs this function. Thus, it n