[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-07 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende I ran this against a large body of our test data, and it seemed to work just fine. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-10 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende Updated. Let me know what you think when you get a chance. Thanks. --- 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 p

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-10 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/1961 @MikeThomsen thanks Mike... I added a couple of comments this morning, not sure if you saw those yet, but I think we're close. --- If your project is set up for it, you can reply to this email and hav

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-17 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende Ok, should be good to go now. --- 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 ena

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-26 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende Saw on the mailing list that you've been away for a little while. Any chance you're back and ready to take a look? --- If your project is set up for it, you can reply to this email and ha

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-07-27 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/1961 @MikeThomsen I should be able to take a look early next week --- 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 hav

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-08-02 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 @bbende Done. --- 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 i

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-08-03 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/1961 Thanks @MikeThomsen , two minor issues while testing this... 1) There is a try-with-resource block that creates the RecordReader, and as part of that line it also creates an InputStream for the

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-08-03 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/1961 Looks good to me. --- 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,

[GitHub] nifi issue #1961: NIFI-4024 Added org.apache.nifi.hbase.PutHBaseRecord

2017-08-04 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/1961 Merged to master, thanks! --- 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 s