[GitHub] incubator-hivemall issue #27: [HIVEMALL-36] Refactor each_top_k

2017-01-26 Thread coveralls
Github user coveralls commented on the issue:

https://github.com/apache/incubator-hivemall/pull/27
  

[![Coverage 
Status](https://coveralls.io/builds/9851772/badge)](https://coveralls.io/builds/9851772)

Coverage increased (+0.01%) to 35.842% when pulling 
**4039f9e17d20f8a8fd804da461e0e2e63e3a6b90 on maropu:HIVEMALL-36** into 
**79f92f4f2c6458ccf4133744a1654a46cbbb56cc on apache:master**.



---
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-hivemall issue #27: [HIVEMALL-36] Refactor each_top_k

2017-01-25 Thread maropu
Github user maropu commented on the issue:

https://github.com/apache/incubator-hivemall/pull/27
  
okay, I'll merge this, then I'll check the OOM issue in follow-up 
activities. 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 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-hivemall issue #27: [HIVEMALL-36] Refactor each_top_k

2017-01-25 Thread myui
Github user myui commented on the issue:

https://github.com/apache/incubator-hivemall/pull/27
  
@maropu LGTM. Please merge this PR and close the JIRA ticket.

BTW, `HiveUdfWithFeatureSuite` causes OOM again.


---
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-hivemall issue #27: [HIVEMALL-36] Refactor each_top_k

2017-01-25 Thread coveralls
Github user coveralls commented on the issue:

https://github.com/apache/incubator-hivemall/pull/27
  

[![Coverage 
Status](https://coveralls.io/builds/9849255/badge)](https://coveralls.io/builds/9849255)

Coverage increased (+0.3%) to 36.14% when pulling 
**98ff0b90e0d47dc1695869a1347b276fac0881a0 on maropu:HIVEMALL-36** into 
**4688494418693de5b08dc0a4bd22685dbcf0b876 on apache:master**.



---
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.
---