Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2736#discussion_r190596306
--- Diff:
nifi-nar-bundles/nifi-standard-services/nifi-record-serialization-service-api/src/main/java/org/apache/nifi/serialization/RecordSetWriterFactory.java
---
@@ -76,5 +78,23 @@
* @return a RecordSetWriter that can write record sets to an
OutputStream
* @throws IOException if unable to read from the given InputStream
*/
- RecordSetWriter createWriter(ComponentLog logger, RecordSchema schema,
OutputStream out) throws SchemaNotFoundException, IOException;
--- End diff --
Instead of changing the interface method, wouldn't it be best to add a new
method? I know we already did this kind of change in the past but it's going to
break custom processors that would use the Record API. If we do that, could be
good to keep it in mind and mention it in the release/upgrade note.
---
