[GitHub] incubator-hivemall issue #114: [HIVEMALL-138-2] refactored to_ordered_map & ...

2017-09-13 Thread coveralls 
Github user coveralls commented on the issue:

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

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

Coverage increased (+0.3%) to 40.536% when pulling 
**df39bd5c8254db064ea2f54e7fce6d5e3f363961 on myui:HIVEMALL-138-2** into 
**3804789168dab5c5d43aac1fd4000e07688c6a06 on apache:master**.



---

[GitHub] incubator-hivemall issue #114: [HIVEMALL-138-2] refactored to_ordered_map & ...

2017-09-13 Thread myui 
Github user myui commented on the issue:

https://github.com/apache/incubator-hivemall/pull/114
  
@takuti reverted some changes by me and merged.


---

[GitHub] incubator-hivemall issue #114: [HIVEMALL-138-2] refactored to_ordered_map & ...

2017-09-12 Thread myui 
Github user myui commented on the issue:

https://github.com/apache/incubator-hivemall/pull/114
  
oops... I forget that default constructor is mandatory in UDAF.

```
Caused by: java.lang.RuntimeException: java.lang.NoSuchMethodException: 
hivemall.tools.map.UDAFToOrderedMap$DescendingMapEvaluator.()
at 
org.apache.hadoop.util.ReflectionUtils.newInstance(ReflectionUtils.java:134)
at 
org.apache.hadoop.hive.ql.plan.AggregationDesc.getGenericUDAFEvaluator(AggregationDesc.java:92)
at 
org.apache.hadoop.hive.ql.exec.GroupByOperator.initializeOp(GroupByOperator.java:350)
at 
org.apache.hadoop.hive.ql.exec.Operator.initialize(Operator.java:366)
at 
org.apache.hadoop.hive.ql.exec.Operator.initialize(Operator.java:556)
at 
org.apache.hadoop.hive.ql.exec.Operator.initializeChildren(Operator.java:508)
at 
org.apache.hadoop.hive.ql.exec.Operator.initialize(Operator.java:376)
at 
org.apache.hadoop.hive.ql.exec.Operator.initialize(Operator.java:556)
at 
org.apache.hadoop.hive.ql.exec.Operator.initializeChildren(Operator.java:508)
at 
org.apache.hadoop.hive.ql.exec.Operator.initialize(Operator.java:376)
at 
org.apache.hadoop.hive.ql.exec.MapOperator.initializeMapOperator(MapOperator.java:501)
at 
org.apache.hadoop.hive.ql.exec.tez.MapRecordProcessor.init(MapRecordProcessor.java:317)
... 15 more
Caused by: java.lang.NoSuchMethodException: 
hivemall.tools.map.UDAFToOrderedMap$DescendingMapEvaluator.()
at java.lang.Class.getConstructor0(Class.java:3082)
at java.lang.Class.getDeclaredConstructor(Class.java:2178)
at 
org.apache.hadoop.util.ReflectionUtils.newInstance(ReflectionUtils.java:128)
... 26 more
```


---

[GitHub] incubator-hivemall issue #114: [HIVEMALL-138-2] refactored to_ordered_map & ...

2017-09-12 Thread myui 
Github user myui commented on the issue:

https://github.com/apache/incubator-hivemall/pull/114
  
Ah... It was Hive v2.3.0 bug and not related to estimate().

https://issues.apache.org/jira/browse/HIVE-17406


---

[GitHub] incubator-hivemall issue #114: [HIVEMALL-138-2] refactored to_ordered_map & ...

2017-09-12 Thread myui 
Github user myui commented on the issue:

https://github.com/apache/incubator-hivemall/pull/114
  
```
FAILED: IllegalArgumentException Size requested for unknown type: 
hivemall.tools.list.UDAFToOrderedList$UDAFToOrderedListEvaluator$AbstractQueueHandler
```

failed in Hive v2.3.0 because UDAF estimate() is lacking.


---

[GitHub] incubator-hivemall issue #114: [HIVEMALL-138-2] refactored to_ordered_map & ...

2017-09-12 Thread myui 
Github user myui commented on the issue:

https://github.com/apache/incubator-hivemall/pull/114
  
@takuti applied some refactoring.


---