[GitHub] incubator-madlib issue #165: Release 1.12: Version numbering and upgrade rel...

2017-08-10 Thread asfgit
Github user asfgit commented on the issue:

https://github.com/apache/incubator-madlib/pull/165
  

Refer to this link for build results (access rights to CI server needed): 
https://builds.apache.org/job/madlib-pr-build/153/



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #161: Docs: Update BFS design docs

2017-08-10 Thread asfgit
Github user asfgit commented on the issue:

https://github.com/apache/incubator-madlib/pull/161
  

Refer to this link for build results (access rights to CI server needed): 
https://builds.apache.org/job/madlib-pr-build/152/



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib pull request #165: Release 1.12: Version numbering and upgr...

2017-08-10 Thread orhankislal
GitHub user orhankislal opened a pull request:

https://github.com/apache/incubator-madlib/pull/165

Release 1.12: Version numbering and upgrade related changes

Updates the version number to 1.12 for the release candidate.
Updates the changelists and other related files for upgrade.
Note that upgrade is not supported from versions prior to 1.9.1.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/orhankislal/incubator-madlib 
release/upgrade_1.12

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-madlib/pull/165.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #165


commit fef25344a29a2e7b5a0c8a78baed3abd93e9ec80
Author: Orhan Kislal 
Date:   2017-08-10T23:53:36Z

Release 1.12: Version numbering and upgrade related changes

Updates the version number to 1.12 for the release candidate.
Updates the changelists and other related files for upgrade.
Note that upgrade is not supported from versions prior to 1.9.1.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #164: Elastic Net: Fix normalization issue

2017-08-10 Thread asfgit
Github user asfgit commented on the issue:

https://github.com/apache/incubator-madlib/pull/164
  

Refer to this link for build results (access rights to CI server needed): 
https://builds.apache.org/job/madlib-pr-build/151/



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Jenkins madlib-master-build failed

2017-08-10 Thread Ed Espino
FYI: The manually triggered Jenkins master build passed:
https://builds.apache.org/view/M-R/view/MADlib/job/madlib-master-build/80/

-=e

On Thu, Aug 10, 2017 at 4:14 PM, Ed Espino  wrote:

> Not sure what caused the MADlib master build to fail (git clone issue?). I
> have re-triggered it and it is beyond the previous failure point.
>
> -=e
>
> Here is the failure for future reference (https://builds.apache.org/
> view/M-R/view/MADlib/job/madlib-master-build/79/console):
>
> Checking out Revision 67b69eb8a5eec1ff5d4b947eabb90970d66b2ac5
> (refs/remotes/origin/master)
> Commit message: "MADLIB-1133. TLP graduation - remove references to
> "incubating"."
>  > git config core.sparsecheckout # timeout=10
>  > git checkout -f 67b69eb8a5eec1ff5d4b947eabb90970d66b2ac5
>  > git rev-list 0dc2df94358bb2ec3fd85865a6d53ae7cbde0226 # timeout=10
> Extended Email Publisher is currently disabled in project settings
> FATAL: Unable to produce a script file
> java.io.IOException: Permission denied
> at java.io.UnixFileSystem.createFileExclusively(Native Method)
> at java.io.File.createTempFile(File.java:2024)
> at hudson.FilePath$17.invoke(FilePath.java:1373)
> at hudson.FilePath$17.invoke(FilePath.java:1363)
> at hudson.FilePath$FileCallableWrapper.call(FilePath.java:2739)
> at hudson.remoting.UserRequest.perform(UserRequest.java:153)
> at hudson.remoting.UserRequest.perform(UserRequest.java:50)
> at hudson.remoting.Request$2.run(Request.java:336)
> at hudson.remoting.InterceptingExecutorService$1.call(
> InterceptingExecutorService.java:68)
> at java.util.concurrent.FutureTask.run(FutureTask.java:266)
> at java.util.concurrent.ThreadPoolExecutor.runWorker(
> ThreadPoolExecutor.java:1142)
> at java.util.concurrent.ThreadPoolExecutor$Worker.run(
> ThreadPoolExecutor.java:617)
> at java.lang.Thread.run(Thread.java:748)
> Caused: java.io.IOException: Failed to create a temporary directory in /tmp
> at hudson.FilePath$17.invoke(FilePath.java:1375)
> at hudson.FilePath$17.invoke(FilePath.java:1363)
> at hudson.FilePath$FileCallableWrapper.call(FilePath.java:2739)
> at hudson.remoting.UserRequest.perform(UserRequest.java:153)
> at hudson.remoting.UserRequest.perform(UserRequest.java:50)
> at hudson.remoting.Request$2.run(Request.java:336)
> at hudson.remoting.InterceptingExecutorService$1.call(
> InterceptingExecutorService.java:68)
> at java.util.concurrent.FutureTask.run(FutureTask.java:266)
> at java.util.concurrent.ThreadPoolExecutor.runWorker(
> ThreadPoolExecutor.java:1142)
> at java.util.concurrent.ThreadPoolExecutor$Worker.run(
> ThreadPoolExecutor.java:617)
> at java.lang.Thread.run(Thread.java:748)
> at ..remote call to H21(Native Method)
> at hudson.remoting.Channel.attachCallSiteStackTrace(Channel.java:1545)
> at hudson.remoting.UserResponse.retrieve(UserRequest.java:253)
> at hudson.remoting.Channel.call(Channel.java:830)
> at hudson.FilePath.act(FilePath.java:986)
> Caused: java.io.IOException: remote file operation failed:
> /home/jenkins/jenkins-slave/workspace/madlib-master-build at
> hudson.remoting.Channel@4b715ff3:H21
> at hudson.FilePath.act(FilePath.java:993)
> at hudson.FilePath.act(FilePath.java:975)
> at hudson.FilePath.createTextTempFile(FilePath.java:1363)
> Caused: java.io.IOException: Failed to create a temp file on
> /home/jenkins/jenkins-slave/workspace/madlib-master-build
> at hudson.FilePath.createTextTempFile(FilePath.java:1386)
> at hudson.tasks.CommandInterpreter.createScriptFile(
> CommandInterpreter.java:162)
> at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:94)
> at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:66)
> at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
> at hudson.model.AbstractBuild$AbstractBuildExecution.
> perform(AbstractBuild.java:735)
> at hudson.model.Build$BuildExecution.build(Build.java:206)
> at hudson.model.Build$BuildExecution.doRun(Build.java:163)
> at hudson.model.AbstractBuild$AbstractBuildExecution.run(
> AbstractBuild.java:490)
> at hudson.model.Run.execute(Run.java:1735)
> at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
> at hudson.model.ResourceController.execute(ResourceController.java:97)
> at hudson.model.Executor.run(Executor.java:405)
> Build step 'Execute shell' marked build as failure
> Skipped archiving because build is not successful
> Recording test results
> Extended Email Publisher is currently disabled in project settings
> Warning: this build has no associated authentication, so build permissions
> may be lacking, and downstream projects which cannot even be seen by an
> anonymous user will be silently skipped
> Finished: FAILURE
>
>
>
> --
> *Ed Espino*
>



-- 
*Ed Espino*


[GitHub] incubator-madlib pull request #164: Elastic Net: Fix normalization issue

2017-08-10 Thread cooper-sloan
GitHub user cooper-sloan opened a pull request:

https://github.com/apache/incubator-madlib/pull/164

Elastic Net: Fix normalization issue

MADLIB-1094 and MADLIB-1146

avg in psql is numerically unstable
Data scaling was not occuring when
grouping is true.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cooper-sloan/incubator-madlib 
elastic_net_normalization

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-madlib/pull/164.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #164


commit 0b00513bf20e7f0b9032b267472321bd6cfc4355
Author: Cooper Sloan 
Date:   2017-08-10T19:04:04Z

Elastic Net: Fix normalization issue

MADLIB-1094 and MADLIB-1146

avg in psql is numerically unstable
Data scaling was not occuring when
grouping is true.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #162: MLP: Multilayer Perceptron Phase 2

2017-08-10 Thread asfgit
Github user asfgit commented on the issue:

https://github.com/apache/incubator-madlib/pull/162
  

Refer to this link for build results (access rights to CI server needed): 
https://builds.apache.org/job/madlib-pr-build/150/



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #162: MLP: Multilayer Perceptron Phase 2

2017-08-10 Thread orhankislal
Github user orhankislal commented on the issue:

https://github.com/apache/incubator-madlib/pull/162
  
GPDB 5 IC passes as well. 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 if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib pull request #158: MADLIB-1133. TLP graduation - remove ref...

2017-08-10 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-madlib/pull/158


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #158: MADLIB-1133. TLP graduation - remove references...

2017-08-10 Thread asfgit
Github user asfgit commented on the issue:

https://github.com/apache/incubator-madlib/pull/158
  

Refer to this link for build results (access rights to CI server needed): 
https://builds.apache.org/job/madlib-pr-build/149/



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #158: MADLIB-1133. TLP graduation - remove references...

2017-08-10 Thread njayaram2
Github user njayaram2 commented on the issue:

https://github.com/apache/incubator-madlib/pull/158
  
Jenkins ok to test


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #163: MADLIB-1118. Change tolerance to 1e-2 (from 1e-...

2017-08-10 Thread njayaram2
Github user njayaram2 commented on the issue:

https://github.com/apache/incubator-madlib/pull/163
  
Jenkins ok to test


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #158: MADLIB-1133. TLP graduation - remove references...

2017-08-10 Thread edespino
Github user edespino commented on the issue:

https://github.com/apache/incubator-madlib/pull/158
  
Jenkins please retest.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #161: Docs: Update BFS design docs

2017-08-10 Thread asfgit
Github user asfgit commented on the issue:

https://github.com/apache/incubator-madlib/pull/161
  

Refer to this link for build results (access rights to CI server needed): 
https://builds.apache.org/job/madlib-pr-build/147/



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib pull request #162: MLP: Multilayer Perceptron Phase 2

2017-08-10 Thread orhankislal
Github user orhankislal commented on a diff in the pull request:

https://github.com/apache/incubator-madlib/pull/162#discussion_r132541348
  
--- Diff: src/ports/postgres/modules/convex/test/mlp.sql_in ---
@@ -241,9 +247,8 @@ SELECT mlp_predict(
 
 select * from mlp_prediction;
 SELECT assert(
--- Accuracy greater than 90%
-COUNT(*)/150.0 > 0.95,
-'MLP: Accuracy is too low (< 95%). Wrong result.'
+COUNT(*)/150.0 > 0.99,
--- End diff --

Tested on PG 9.4 and the latest commit of GPDB 5 (1 node, 3 segments).
IC passes on Postgres but fails on GPDB5. I tested the code manually and it 
works fine with 148/150 accuracy. I would suggest dropping the assertion 
threshold to 95% to make sure such minute changes do not fail the whole test.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib pull request #162: MLP: Multilayer Perceptron Phase 2

2017-08-10 Thread orhankislal
Github user orhankislal commented on a diff in the pull request:

https://github.com/apache/incubator-madlib/pull/162#discussion_r132541507
  
--- Diff: src/ports/postgres/modules/convex/test/mlp.sql_in ---
@@ -241,9 +247,8 @@ SELECT mlp_predict(
 
 select * from mlp_prediction;
--- End diff --

select * from -> SELECT * FROM


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #161: Docs: Update BFS design docs

2017-08-10 Thread njayaram2
Github user njayaram2 commented on the issue:

https://github.com/apache/incubator-madlib/pull/161
  
Jenkins ok to test


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #163: MADLIB-1118. Change tolerance to 1e-2 (from 1e-...

2017-08-10 Thread njayaram2
Github user njayaram2 commented on the issue:

https://github.com/apache/incubator-madlib/pull/163
  
@iyerr3 Not sure why they are commented out.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


MADlib PR Jenkins jobs

2017-08-10 Thread Ed Espino
It appears a few of the PRs are in a stalled state (Some checks haven’t
completed yet) for the Jenkins "madlib-pr-build" (
https://builds.apache.org/view/M-R/view/MADlib/job/madlib-pr-build/) job.

Any idea what might be going on?

A sample of stalled build check PRs:

https://github.com/apache/incubator-madlib/pull/161
https://github.com/apache/incubator-madlib/pull/163

-=e

-- 
*Ed Espino*


[GitHub] incubator-madlib issue #163: MADLIB-1118. Change tolerance to 1e-2 (from 1e-...

2017-08-10 Thread edespino
Github user edespino commented on the issue:

https://github.com/apache/incubator-madlib/pull/163
  
@iyerr3 - Thanks for the review. Very much appreciated. Additionally, I 
should have reviewed the `install-check` options. Thanks for the info.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-madlib issue #163: MADLIB-1118. Change tolerance to 1e-2 (from 1e-...

2017-08-10 Thread iyerr3
Github user iyerr3 commented on the issue:

https://github.com/apache/incubator-madlib/pull/163
  
The change looks good. 

Few comments: 
- An alternative to changing the threshold is to reduce the max number of 
iterations. Even with the lower threshold, we're not necessarily guaranteed 
quicker completion. 
- The log file can be accessed even with the test passing by adding `-vl` 
option to the `madpack install-check` command. The options indicate `-v: 
Verbose` and `-l: Keep logs`. 
- The install-check itself also provides the run time for execution of the 
whole file. However, `\timing` is needed if run time for individual queries is 
desired. 

@njayaram2 Any idea why the asserts on `log_likelihood` are commented out? 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: [VOTE]: MADlib repo(s) migration

2017-08-10 Thread ChenLiang Wang
1

On 08/10/2017 05:47 AM, Orhan Kislal wrote:
> 1
> 
> Orhan Kislal
> 
> On Wed, Aug 9, 2017 at 2:32 PM, Nandish Jayaram  wrote:
> 
>> Hi All,
>>
>> With MADlib's graduation to TLP, it's time to migrate its github
>> repos from `*incubator-madlib*` to `*madlib*`. We will have to open
>> an Apache Infrastructure ticket to request this move for the following
>> repos (along with other stuff like wiki, jenkins etc):
>> https://git1-us-west.apache.org/repos/asf?p=incubator-madlib.git
>>  (Read/Write)
>> https://github.com/apache/incubator-madlib (Github mirror- read only)
>> https://git1-us-west.apache.org/repos/asf?p=incubator-madlib-site.git
>> https://github.com/apache/incubator-madlib-site (GitHub mirror)
>>
>> There are two ways to go about this, and the Infra ticket has to be
>> raised accordingly.
>> 1) Just maintain the current set-up, but have the repos renamed from
>> incubator-madlib to madlib.
>> 2) Use Gitbox to enable github repo as a R/W repo and not just read-only.
>> Check this email (
>> https://mail-archives.apache.org/mod_mbox/incubator-madlib-
>> dev/201708.mbox/%3cCA+ULb+vP0ViWH4Nc=4eaXvbT0KOmeFtQzp4eAa3p0fKPP7c
>> 8...@mail.gmail.com%3e)
>> for further information.
>>
>> Please vote you preference and we can decide to move accordingly.
>>
>> NJ
>>
>