Re: Stability of the Record API

2018-08-09 Thread Bryan Bende
I don't think there are any stability issues with the record API, it is definitely recommended to use the record approach where it makes sense. That comment was probably put there on the first release and never removed, and now it has been 4-5 releases later. As a general comment to APIs, the rec

Re: Stability of the Record API

2018-08-09 Thread Mike Thomsen
I think that comment is no longer valid. Heck PutHBaseRecord started as part of a project at my company in early 2017 and we found it perfectly stable back then. On Wed, Aug 8, 2018 at 11:46 PM Otto Fowler wrote: > I’m seeing > > https://github.com/apache/nifi/blob/master/nifi-commons/nifi-record

Stability of the Record API

2018-08-08 Thread Otto Fowler
I’m seeing https://github.com/apache/nifi/blob/master/nifi-commons/nifi-record/src/main/java/org/apache/nifi/serialization/RecordReader.java#L34 being quoted as a reason to NOT build Record based processors but instead stick with the original Processor api. Yet, on list and on hipchat and in pr’s