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

2016-07-29 Thread Priya Ranjan
Thanks Greg, but not as a binding but a substitute for C++ code, as this code is untrusted there could be an exception bringing down the entire server. Rustlang guarantees memory safety. Regards On Sat, 30 Jul 2016 11:09 Greg Chase wrote: > Hi Rangan, > Its nice to hear from you. > > I'll let o

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

2016-07-29 Thread Greg Chase
Hi Rangan, Its nice to hear from you. I'll let other folks of the community speak about language bindings. But, and important thing to understand about MADlib in its current capabilities is that it is developed to work in-database in a variety of PostgreSQL-derived database platforms. To date, we

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

2016-07-29 Thread Priya Ranjan
Hello Greg Unfortunately I wasn't able to join in but had one question whether you guys are looking at Rustlang as it guarantees memory safety with high performance for building core MADlib libraries or embedding it in your existing codebase. Wanted to know your thoughts. Regards Ranjan On 28 J

[GitHub] incubator-madlib pull request #61: Update README.md

2016-07-29 Thread fmcquillan99
GitHub user fmcquillan99 opened a pull request: https://github.com/apache/incubator-madlib/pull/61 Update README.md Fixed a bunch of broken links You can merge this pull request into a Git repository by running: $ git pull https://github.com/fmcquillan99/incubator-madlib patch-

[GitHub] incubator-madlib issue #54: Pivoting: Phase 2

2016-07-29 Thread orhankislal
Github user orhankislal commented on the issue: https://github.com/apache/incubator-madlib/pull/54 I see what you mean, should be easy enough. --- 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

[GitHub] incubator-madlib issue #54: Pivoting: Phase 2

2016-07-29 Thread iyerr3
Github user iyerr3 commented on the issue: https://github.com/apache/incubator-madlib/pull/54 Even for the other two, I suggest keeping the static parts all together with only the dynamic part created in the loops. That avoids having the `SELECT` clause separated from the `FROM` by 70

Re: Missing Wiki tab on apache incubator madlib github repo

2016-07-29 Thread Shubham Sharma
Thanks Frank. On Fri, Jul 29, 2016 at 3:46 PM, Frank McQuillan wrote: > Sorry, that link is wrong. The MADlib wiki is at > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=61319606 > > I'll fix it the link > > Regards, > Frank > > > > > On Fri, Jul 29, 2016 at 3:37 PM, Shubham S

Re: Missing Wiki tab on apache incubator madlib github repo

2016-07-29 Thread Frank McQuillan
Sorry, that link is wrong. The MADlib wiki is at https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=61319606 I'll fix it the link Regards, Frank On Fri, Jul 29, 2016 at 3:37 PM, Shubham Sharma wrote: > Hi, > > I was going through the README.md present at github repository > h

[GitHub] incubator-madlib pull request #60: Path: Proposed fix for JIRA MADLIB-1012

2016-07-29 Thread mktal
Github user mktal closed the pull request at: https://github.com/apache/incubator-madlib/pull/60 --- 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 if the feature

[GitHub] incubator-madlib issue #60: Path: Proposed fix for JIRA MADLIB-1012

2016-07-29 Thread mktal
Github user mktal commented on the issue: https://github.com/apache/incubator-madlib/pull/60 Great. I will go ahead and close this one. --- 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 fe

[GitHub] incubator-madlib issue #54: Pivoting: Phase 2

2016-07-29 Thread orhankislal
Github user orhankislal commented on the issue: https://github.com/apache/incubator-madlib/pull/54 Out of 5 sql queries, I reduced the first 3 to single lines as you suggested. For the other two I would rather keep them broken into a list since they are created inside the 4 level loop

Missing Wiki tab on apache incubator madlib github repo

2016-07-29 Thread Shubham Sharma
Hi, I was going through the README.md present at github repository https://github.com/apache/incubator-madlib, the madlib wiki hyperlink appears to be broken. The only wiki article I can find on google was on https://github.com/madlib/madlib/wiki(I think this is the old repository). I was trying t

[GitHub] incubator-madlib issue #54: Pivoting: Phase 2

2016-07-29 Thread iyerr3
Github user iyerr3 commented on the issue: https://github.com/apache/incubator-madlib/pull/54 Could you please change all dynamically created SQL strings to have the query structure as a single string with variables filling in the dynamic entries? It's much easier to understand the qu

[GitHub] incubator-madlib issue #60: Path: Proposed fix for JIRA MADLIB-1012

2016-07-29 Thread iyerr3
Github user iyerr3 commented on the issue: https://github.com/apache/incubator-madlib/pull/60 Yes the expected behavior is that all standard postgresql identifiers are allowed. On Fri, Jul 29, 2016 at 3:15 PM, Frank McQuillan wrote: > I guess the wider questio

[GitHub] incubator-madlib issue #60: Path: Proposed fix for JIRA MADLIB-1012

2016-07-29 Thread fmcquillan99
Github user fmcquillan99 commented on the issue: https://github.com/apache/incubator-madlib/pull/60 I guess the wider question is are we supporting standard postgres identifiers for symbols? https://www.postgresql.org/docs/8.2/static/plpgsql-statements.html So far I have

[GitHub] incubator-madlib issue #60: Path: Proposed fix for JIRA MADLIB-1012

2016-07-29 Thread iyerr3
Github user iyerr3 commented on the issue: https://github.com/apache/incubator-madlib/pull/60 👍 I guess we were looking at the same thing. I already pushed the fix in da582e8. You can close this one when you get the chance. --- If your project is set up for it, you can reply to

[GitHub] incubator-madlib pull request #60: Path: Proposed fix for JIRA MADLIB-1012

2016-07-29 Thread mktal
GitHub user mktal opened a pull request: https://github.com/apache/incubator-madlib/pull/60 Path: Proposed fix for JIRA MADLIB-1012 You can merge this pull request into a Git repository by running: $ git pull https://github.com/mktal/incubator-madlib bugfix/path_underscore Al

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread orhankislal
Github user orhankislal commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72861263 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72859110 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread orhankislal
Github user orhankislal commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72858556 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread orhankislal
Github user orhankislal commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72857123 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72857563 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72856933 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72856491 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread orhankislal
Github user orhankislal commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72856182 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,

[GitHub] incubator-madlib pull request #54: Pivoting: Phase 2

2016-07-29 Thread iyerr3
Github user iyerr3 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/54#discussion_r72855881 --- Diff: src/ports/postgres/modules/utilities/pivot.py_in --- @@ -58,66 +61,256 @@ def pivot(schema_madlib, source_table, out_table,