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

ASF GitHub Bot commented on STREAMS-141:
----------------------------------------

Github user mfranklin commented on a diff in the pull request:

    https://github.com/apache/incubator-streams/pull/62#discussion_r15872031
  
    --- Diff: 
streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/serializer/StreamsTwitterMapper.java
 ---
    @@ -68,7 +71,14 @@ public StreamsTwitterMapper() {
                     addDeserializer(DateTime.class, new 
StdDeserializer<DateTime>(DateTime.class) {
                         @Override
                         public DateTime deserialize(JsonParser jpar, 
DeserializationContext context) throws IOException, JsonProcessingException {
    -                        return 
TWITTER_FORMAT.parseDateTime(jpar.getValueAsString());
    +                        DateTime result = null;
    +                        try {
    +                            result = 
TWITTER_FORMAT.parseDateTime(jpar.getValueAsString());
    --- End diff --
    
    Can we just update RFC3339Utils to have the twitter format rather than 
forcing a separate mapper?


> StreamsTwitterMapper can't reprocess Streams Outputs
> ----------------------------------------------------
>
>                 Key: STREAMS-141
>                 URL: https://issues.apache.org/jira/browse/STREAMS-141
>             Project: Streams
>          Issue Type: Bug
>            Reporter: Steve Blackmon
>
> Update StreamsTwitterMapper to parse DateTime as Twitter formatted strings OR 
> RFC3339



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to