You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by HeartSaVioR <gi...@git.apache.org> on 2016/10/10 20:02:05 UTC
[GitHub] storm pull request #1734: STORM-2141 UDAFWrapper needs to provide parameter ...
GitHub user HeartSaVioR opened a pull request:
https://github.com/apache/storm/pull/1734
STORM-2141 UDAFWrapper needs to provide parameter information when error invoking method
While developing UDF and UDAF, users should be aware of association between Calcite RelType and Java Type.
Though Calcite validates SQL statement, there seems be the chance for UDAF to be called for unexpected parameter types, and "java.lang.IllegalArgumentException: argument type mismatch" is thrown.
We can grab it and provide expected parameters and actual parameters so that users can investigate why.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/HeartSaVioR/storm STORM-2141
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/1734.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1734
----
commit eef9538f84366e7bc23f57d185b08331891f3bb0
Author: Jungtaek Lim <ka...@gmail.com>
Date: 2016-10-10T19:59:20Z
STORM-2141 UDAFWrapper needs to provide parameter information when error invoking method
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request #1734: STORM-2141 [Storm SQL] UDAFWrapper needs to provid...
Posted by HeartSaVioR <gi...@git.apache.org>.
Github user HeartSaVioR closed the pull request at:
https://github.com/apache/storm/pull/1734
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm issue #1734: STORM-2141 [Storm SQL] UDAFWrapper needs to provide param...
Posted by HeartSaVioR <gi...@git.apache.org>.
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1734
This is against STORM-2200 which is discussed on dev@. Closing.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---