[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 okay, thanks! --- 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 s

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 @wangyum @maropu Thanks. Merged with some modifications. Configuration for spark-1.6 should also be changed. --- If your project is set up for it, you can reply to this email and have your r

[GitHub] incubator-hivemall pull request #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536...

2017-01-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-hivemall/pull/22 --- 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 feat

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 [![Coverage Status](https://coveralls.io/builds/9793022/badge)](https://coveralls.io/builds/9793022) Coverage remained the same at 35.842% when pulling **20c2eea9f181b073e9

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 [![Coverage Status](https://coveralls.io/builds/9793022/badge)](https://coveralls.io/builds/9793022) Coverage remained the same at 35.842% when pulling **20c2eea9f181b073e9

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread wangyum
Github user wangyum commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 I'll try to set 512m. --- 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

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 @myui How about 512m? Sorry, but I forgot why I set that value there though, I checked the spark configure and I found that value is 512m. --- If your project is set up for it, you can rep

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 @maropu It seems GC is still happening for some cases... ``` HivemallFeatureOpsSuite: No output has been received in the last 10m0s, this potentially indicates a stalled build or s

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 [![Coverage Status](https://coveralls.io/builds/9792621/badge)](https://coveralls.io/builds/9792621) Coverage increased (+0.3%) to 36.14% when pulling **37da2269a478d40b53a

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 @myui In running these tests, a spark context and unit tests work in the same JVM. The context uses at most 1g memory by default. So, in some conditions, the unit tests run short of memory,

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 @maropu BTW, why `mvn -q scalastyle:check test -Pspark-2.0" exited with 1` is happening? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 okya, LGTM cc: @myui --- 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 w

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 Oh, I got you. So, how about setting these values along with the spark one (e.g., -mx3g)? See: https://github.com/apache/spark/blob/master/pom.xml#L2066 --- If your project is set up for i

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to -Xmx1536m to av...

2017-01-22 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 TravisCI always pass these tests though, why did you got the error? What's your environment? --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] incubator-hivemall issue #21: [HIVEMALL-29] Add github pull request template

2017-01-22 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/21 @wangyum Thank you for the contribution. Merged with small modifications. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-hivemall pull request #21: [HIVEMALL-29] Add github pull request t...

2017-01-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-hivemall/pull/21 --- 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 feat

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx to 1536 to avoid O...

2017-01-22 Thread myui
Github user myui commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 @maropu Could you take a look at? --- 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 en

[GitHub] incubator-hivemall issue #22: [HIVEMALL-30] Increase -Xmx from 1024 to 1536

2017-01-22 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-hivemall/pull/22 [![Coverage Status](https://coveralls.io/builds/9787706/badge)](https://coveralls.io/builds/9787706) Coverage remained the same at 35.842% when pulling **1d18c7d3f7328f57a2

[GitHub] incubator-hivemall pull request #22: [HIVEMALL-30] Increase -Xmx from 1024 t...

2017-01-22 Thread wangyum
GitHub user wangyum opened a pull request: https://github.com/apache/incubator-hivemall/pull/22 [HIVEMALL-30] Increase -Xmx from 1024 to 1536 ## What changes were proposed in this pull request? Increase -Xmx from 1024 to 1536. ## How was this patch tested?

[GitHub] incubator-hivemall issue #21: [HIVEMALL-29] Add github pull request template

2017-01-22 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/incubator-hivemall/pull/21 [![Coverage Status](https://coveralls.io/builds/9787265/badge)](https://coveralls.io/builds/9787265) Coverage decreased (-0.01%) to 35.831% when pulling **7106b4276e653cad8

[GitHub] incubator-hivemall pull request #21: [HIVEMALL-29] Add github pull request t...

2017-01-22 Thread wangyum
GitHub user wangyum opened a pull request: https://github.com/apache/incubator-hivemall/pull/21 [HIVEMALL-29] Add github pull request template ## What changes were proposed in this pull request? Add github pull request template. ## How was this patch tested?