You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by liu-zhaokun <gi...@git.apache.org> on 2017/08/03 01:17:57 UTC
[GitHub] storm pull request #2259: [STORM-2676] Error class name for log in JsonRecor...
GitHub user liu-zhaokun opened a pull request:
https://github.com/apache/storm/pull/2259
[STORM-2676] Error class name for log in JsonRecordHiveMapper.java.
[https://issues.apache.org/jira/browse/STORM-2676](https://issues.apache.org/jira/browse/STORM-2676)
The class in "LoggerFactory.getLogger()" should be JsonRecordHiveMapper.class.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/liu-zhaokun/storm master08030857
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2259.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 #2259
----
commit 7340590ceb9549eb3fe8bc443ba9576260d337cc
Author: liuzhaokun <li...@zte.com.cn>
Date: 2017-08-03T01:16:02Z
[STORM-2676] Error class name for log in JsonRecordHiveMapper.java.
----
---
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 #2259: [STORM-2676] Error class name for log in JsonRecordHiveMa...
Posted by liu-zhaokun <gi...@git.apache.org>.
Github user liu-zhaokun commented on the issue:
https://github.com/apache/storm/pull/2259
@revans2
OK,thanks for your reply.
---
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 #2259: [STORM-2676] Error class name for log in JsonRecordHiveMa...
Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2259
+1
---
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 #2259: [STORM-2676] Error class name for log in JsonRecordHiveMa...
Posted by liu-zhaokun <gi...@git.apache.org>.
Github user liu-zhaokun commented on the issue:
https://github.com/apache/storm/pull/2259
@HeartSaVioR
Could you have a look if you have some time?
---
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 #2259: [STORM-2676] Error class name for log in JsonRecordHiveMa...
Posted by liu-zhaokun <gi...@git.apache.org>.
Github user liu-zhaokun commented on the issue:
https://github.com/apache/storm/pull/2259
@HeartSaVioR
Thanks for your support.Could you help me merge it if you have some time?
---
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 #2259: [STORM-2676] Error class name for log in JsonRecordHiveMa...
Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2259
@liu-zhaokun we have a policy that we need to wait 24 hours before merging a change, even a trivial one like this to give all devs time to review and comment.
---
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 #2259: [STORM-2676] Error class name for log in JsonRecordHiveMa...
Posted by HeartSaVioR <gi...@git.apache.org>.
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2259
+1
---
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 #2259: [STORM-2676] Error class name for log in JsonRecor...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2259
---
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.
---