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

Joseph Witt commented on NIFI-855:
----------------------------------

Few more things to work out:

- The validation process is quite nice.  But it appears it should be valid for 
users to enter multiple locations as is shown here:
https://dev.twitter.com/streaming/overview/request-parameters#locations

We should validate that zero or more swLon,swLat,neLon,neLat groupings are 
present.  We should extract each group and add the location and iterate from 
there.  Rather than relying on exception handling for number format exceptions 
recommend instead doing a regex based check.  This would be good anyway as it 
means you could validate the precision makes sense for lats/longs.

Also would be good to make the logic for handling those lat/longs testable and 
put them in a unit test.

Thanks
Joe

> Add location bounding box filter to twitter processor
> -----------------------------------------------------
>
>                 Key: NIFI-855
>                 URL: https://issues.apache.org/jira/browse/NIFI-855
>             Project: Apache NiFi
>          Issue Type: New Feature
>          Components: Examples
>    Affects Versions: 0.4.0, 0.2.1
>            Reporter: Chris Mangold
>              Labels: patch
>             Fix For: 0.3.0
>
>         Attachments: nifi-855.patch
>
>   Original Estimate: 72h
>  Remaining Estimate: 72h
>
> Extend existing Twitter NIFI processor to support bounding box filter.



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

Reply via email to