[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-08-09 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/spark/pull/14422 ok --- 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 featur

[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-08-07 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14422 @xubo245 you need to close the issues, we can't do so directly. --- 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 no

[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-08-07 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/spark/pull/14422 @srowen sorry, please close the issue. I will learning more before next PR. The PR is only because breeze have the function. In spark ,there is no use for them. Could you tell me some issue for

[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-07-31 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14422 Please don't open other PRs for related changes. Usually, you also need a JIRA. Although I understand the use for these methods, is there any use for them in Spark? we probably wouldn't add these thi

[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-07-30 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/spark/pull/14422 @HyukjinKwon Thank you. This is my first time to push request to spark, Sorrry, I will follow the https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark later. --- If your p

[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-07-30 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14422 I guess it might be ncer if the contributions you make follow https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark.just like other contributions. --- If your project is set

[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-07-30 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/spark/pull/14422 I add test : https://github.com/apache/spark/pull/14424 --- 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] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-07-30 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14422 Doesn't that need a JIRA, test and PR descriptio maybe? --- 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 h

[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-07-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14422 Can one of the admins verify this patch? --- 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 feat

[GitHub] spark issue #14422: Add rand(numRows: Int, numCols: Int) functions

2016-07-30 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/spark/pull/14422 we can use it to replacebreeze.linalg.DenseMatrix.rand(numRows: Int, numCols: Int) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If