[ https://issues.apache.org/jira/browse/NIFI-5122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16456820#comment-16456820 ]
ASF GitHub Bot commented on NIFI-5122: -------------------------------------- Github user mattyb149 commented on a diff in the pull request: https://github.com/apache/nifi/pull/2663#discussion_r184757401 --- Diff: nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting/SiteToSiteBulletinReportingTask.java --- @@ -76,10 +81,16 @@ private volatile long lastSentBulletinId = -1L; + public SiteToSiteBulletinReportingTask() throws IOException { + final InputStream schema = getClass().getClassLoader().getResourceAsStream("schema-bulletins.avsc"); + recordSchema = AvroTypeUtil.createSchema(new Schema.Parser().parse(schema)); + } + @Override protected List<PropertyDescriptor> getSupportedPropertyDescriptors() { final List<PropertyDescriptor> properties = new ArrayList<>(super.getSupportedPropertyDescriptors()); properties.add(PLATFORM); + properties.add(RECORD_WRITER); --- End diff -- Is the intent here to allow all S2S Reporting Tasks to use the record writer if they want, or do we want to make it "mandatory" by including it in the properties from the abstract base class? > Add record writer to S2S Reporting Tasks > ---------------------------------------- > > Key: NIFI-5122 > URL: https://issues.apache.org/jira/browse/NIFI-5122 > Project: Apache NiFi > Issue Type: Improvement > Components: Extensions > Reporter: Pierre Villard > Assignee: Pierre Villard > Priority: Major > > Just like we have the option to specify a record writer for the new Site To > Site Metrics Reporting Task, there should be the possibility to specify an > optional record writer for the other S2S reporting tasks. -- This message was sent by Atlassian JIRA (v7.6.3#76005)