You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Frank Yellin (JIRA)" <ji...@apache.org> on 2016/08/03 23:50:20 UTC
[jira] [Created] (BEAM-524) Description of "type" argument in
Aggregator is incorrect
Frank Yellin created BEAM-524:
---------------------------------
Summary: Description of "type" argument in Aggregator is incorrect
Key: BEAM-524
URL: https://issues.apache.org/jira/browse/BEAM-524
Project: Beam
Issue Type: Bug
Reporter: Frank Yellin
Priority: Minor
Two problems with documentation for "type" argument.
Trivial: Remove "by default". This phrase implies that there are other alternatives besides what is listed. There aren't.
Non trivial. The documentation says "types appropriate to the combine_fn" are accepted. I tried
Accumulator("foo", max, datetime.datetime)
This failed even though "datetime.datetime" is a perfectly reasonable type to want to take the max of. (I wanted to know precisely when the last job finished.)
Either the documentation needs to be changed to specify that max/min only apply to numeric types, or the code needs to be changed to allow other uses of min and max.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)