[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-02 Thread decibel
Github user decibel commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65610699 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-02 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65604154 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-02 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65597573 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-02 Thread decibel
Github user decibel commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65589213 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-02 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65578128 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

Re: [GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-02 Thread Jim Nasby
I quickly tried the query with min_time proposed in MADLIB-1002, and it seems to be missing one of the requirements. What is it missing?

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65457433 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65455496 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-01 Thread decibel
Github user decibel commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65454766 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65446205 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-01 Thread njayaram2
Github user njayaram2 commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65445316 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-01 Thread decibel
Github user decibel commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65437304 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] incubator-madlib pull request #44: Feature: Sessionize funtion

2016-06-01 Thread decibel
Github user decibel commented on a diff in the pull request: https://github.com/apache/incubator-madlib/pull/44#discussion_r65424347 --- Diff: src/ports/postgres/modules/utilities/sessionize.py_in --- @@ -0,0 +1,101 @@ +# Licensed to the Apache Software Foundation (ASF) under