You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Anton Vinogradov (JIRA)" <ji...@apache.org> on 2015/10/01 12:19:26 UTC

[jira] [Comment Edited] (IGNITE-530) Implement IgniteTwitterStreamer to stream data from Twitter feeds

    [ https://issues.apache.org/jira/browse/IGNITE-530?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14939496#comment-14939496 ] 

Anton Vinogradov edited comment on IGNITE-530 at 10/1/15 10:18 AM:
-------------------------------------------------------------------

Lalit,

Thanks for fixing pull request,

Unfortunately code still not ready to be reviewed.
1) Redundant newlines should be removed. At code and imports.
2) Newlines should be added where it necessary:
for example:
Map<String, String> value = Maps.newHashMap();
value.put(String.valueOf(status.getId()), status.getText());
return value;
3) Some methods have no javadoc.
4) Javadoc sentences should be finished with dot. Javadoc should be started wlth uppercase letter.
5) @Override should be located according to Coding Guidelines.

Please recheck whole code matches Coding Guidelines.


was (Author: avinogradov):
Sorry, but I see no changes at https://github.com/apache/ignite/pull/106

> Implement IgniteTwitterStreamer to stream data from Twitter feeds
> -----------------------------------------------------------------
>
>                 Key: IGNITE-530
>                 URL: https://issues.apache.org/jira/browse/IGNITE-530
>             Project: Ignite
>          Issue Type: Sub-task
>          Components: streaming
>            Reporter: Dmitriy Setrakyan
>            Assignee: Lalit Kumar Jha
>
> We have {{IgniteDataStreamer}} which is used to load data into Ignite under high load. It was previously named {{IgniteDataLoader}}, see ticket IGNITE-394.
> We should create {{IgniteTwitterStreamer}} which will consume messages from Twitter and stream them into Ignite caches.
> More details to follow, but to the least we should be able to:
> * Convert Twitter data to Ignite data using an optional pluggable converter.
> * Specify the cache name for the Ignite cache to load data into.
> * Specify other flags available on {{IgniteDataStreamer}} class.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)