Github user decibel commented on the pull request:
https://github.com/apache/incubator-madlib/pull/41#issuecomment-216661096
Ahh, I didn't realize docs went with the SQL (I'm new around here...).
Is the generated output of the docs by chance available anywhere?
It would be nice if there were SRF (set returning function) versions of
the functions that gave you the output directly, but maybe that's not
the MADlib way of doing things. If those functions existed then the
versions that created output tables would just be simple wrappers around
them. Ideally the SRFs would be plpgsql instead of plpython, as that
would eliminate data marshaling between plpython and Postgres. I'd
definitely solicit feedback from the community before coding that up
though, as it may not fit in with community standards.
Personally, I think the unit test needs to actually verify that the
correct results are being generated, *especially* with data that
contains nulls.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
