+1 (binding)

- Verified signatures and hashes.
- Built NiFi on Ubuntu 20.04 with Java 8 (Adoptium Temurin
1.8.0_352-b08), Java 11 (Adoptium Temurin 11.0.17+8) and Java 17
(Adoptium Temurin 17.0.5+8).
- Ran NiFi on the java versions above.
- Ran flows for testing:
-- new Snowflake processors (NIFI-10370)
-- new Iceberg processor (NIFI-10442)
-- MQTT failover (NIFI-10543)
-- other smaller changes (NIFI-10349, NIFI-10656, NIFI-10317)

Found some issues but these are not blockers. Jiras created:
- NIFI-10883: SnowflakeComputingConnectionPool fails on Java 17
- NIFI-10884: Conflict resolution in PutAzureDataLakeStorage should
log the target filename
- NIFI-10885: ConsumeMQTT should stop client threads

Thanks for RMing Joe!

Regards,
Peter Turcsanyi

On Mon, Nov 28, 2022 at 4:53 AM Mark Bean <mark.o.b...@gmail.com> wrote:
>
> +1 (non-binding)
>
> Verified checksums and signatures.
> Performed full build with -Pcontrib-check using OpenJDK 1.8.0_352, 11.0.17,
> 17.0.5
> Ran several relatively simple flows in non-cluster mode only. No issues
> observed.
>
> I don't believe this is worth holding up the 1.19.0 release, but while
> everything worked as expected with Java 8 and 11, there were some issues
> with Java 17 in generating documentation for several of the "scripted"
> processors, controller services and a reporting task. The following is from
> the nifi-app.log. And, in the UI, the documentation was not full/complete
> for these processors.
>
> 2022-11-27 20:50:27,703 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class
> org.apache.nifi.processors.script.InvokeScriptedProcessor]
> 2022-11-27 20:50:28,243 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class org.apache.nifi.processors.script.ExecuteScript]
> 2022-11-27 20:50:28,311 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class
> org.apache.nifi.record.sink.script.ScriptedRecordSink]
> 2022-11-27 20:50:28,313 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class
> org.apache.nifi.record.script.ScriptedRecordSetWriter]
> 2022-11-27 20:50:28,315 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class org.apache.nifi.lookup.script.ScriptedLookupService]
> 2022-11-27 20:50:28,323 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class
> org.apache.nifi.lookup.script.SimpleScriptedLookupService]
> 2022-11-27 20:50:28,326 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class
> org.apache.nifi.rules.handlers.script.ScriptedActionHandler]
> 2022-11-27 20:50:28,327 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class org.apache.nifi.record.script.ScriptedReader]
> 2022-11-27 20:50:28,328 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class
> org.apache.nifi.rules.engine.script.ScriptedRulesEngine]
> 2022-11-27 20:50:28,521 WARN [main]
> o.apache.nifi.documentation.DocGenerator Documentation generation failed:
> Component Class [class
> org.apache.nifi.reporting.script.ScriptedReportingTask]
>
> On Sun, Nov 27, 2022 at 4:35 PM Nandor Soma Abonyi
> <nsabo...@icloud.com.invalid> wrote:
>
> > +1 (non-binding)
> >
> > I’ve found two minor issues. I believe none of them is a blocker, but we
> > should follow up on them. Marked them with ***.
> >
> > Went through the release helper guide on:
> > Apache Maven 3.8.6 (84538c9988a25aec085021c365c560670ad80f63)
> > Java version: 1.8.0_332, vendor: Temurin
> > Default locale: en_US, platform encoding: UTF-8
> > OS name: "mac os x", version: "10.16", arch: "x86_64", family: "mac"
> >
> > Imported flows from NiFi registry for these issues and verified them:
> > - NIFI-10760 - Add Api key authentication option to
> > ElasticSearchClientServiceImpl
> >     - *** A validation has been removed, which prevented using Basic and
> > Api Key properties together.
> >              If I set Username/Password and hide them by selecting Api Key
> > Authorization Scheme, the client will still use those credentials. If I set
> > Api Key, the client will add both credentials as a request header.
> >              Opened NIFI-10880 for further discussion.
> > - NIFI-10668 - Persist proxy password in remote process group
> > - NIFI-10656 - Log ignored event with info instead of warning in PutADLS
> > - NIFI-10650 - Fix demarcator is appended to the end of the FlowFile's
> > content by ConsumeMQTT
> > - NIFI-10644 - Add Message Demarcator-style processing in PublishMQTT
> > - NIFI-10543 - Support broker failover in MQTT processors
> > - NIFI-10491 - PutAzureBlobStorage_v12 should have Conflict Resolution
> > Strategy
> > - NIFI-10317 - NullPointerException if AMQP header value is null
> >
> > *** Initially, I wanted to build NiFi in a docker container but
> > experienced intermittent test failures. Most of the time, an 5 sec Test
> > timeout caused issues in TestStandardFlowFileQueue.
> >       I don’t know why such a timeout was used, but I opened NIFI-10881
> > for further discussion.
> >
> > Thanks for RM'ing Joe!
> > Regards,
> > Soma
> >
> > > On Nov 23, 2022, at 4:58 PM, Joe Witt <joew...@apache.org> wrote:
> > >
> > > Hello,
> > >
> > > I am pleased to be calling this vote for the source release of Apache
> > NiFi
> > > 1.19.0.
> > >
> > > The source zip, including signatures, digests, etc. can be found at:
> > > https://repository.apache.org/content/repositories/orgapachenifi-1216
> > >
> > > The source being voted upon and the convenience binaries can be found at:
> > > https://dist.apache.org/repos/dist/dev/nifi/nifi-1.19.0/
> > >
> > > A helpful reminder on how the release candidate verification process
> > works:
> > >
> > https://cwiki.apache.org/confluence/display/NIFI/How+to+help+verify+an+Apache+NiFi+release+candidate
> > >
> > > The Git tag is nifi-1.19.0-RC1
> > > The Git commit ID is ec87bf93add2f645d2ea426002e0c24db54614ff
> > >
> > https://gitbox.apache.org/repos/asf?p=nifi.git;a=commit;h=ec87bf93add2f645d2ea426002e0c24db54614ff
> > >
> > > Checksums of nifi-1.19.0-source-release.zip:
> > > SHA256: 909c4fce81b305af955540f83115474d139c132e886847df7b76e94efd3d4641
> > > SHA512:
> > >
> > b3ba95984841140aa2a9a2ea471dd446d6fbd71dba320f723839e0938a7873f847e415a88b7c38e59907ad051c896949f0c7093c6d5e6b5db549de929ec32c6e
> > >
> > > Release artifacts are signed with the following key:
> > > https://people.apache.org/keys/committer/joewitt.asc
> > >
> > > KEYS file available here:
> > > https://dist.apache.org/repos/dist/release/nifi/KEYS
> > >
> > > 221 issues were closed/resolved for this release:
> > >
> > https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12316020&version=12352150
> > >
> > > Release note highlights can be found here:
> > >
> > https://cwiki.apache.org/confluence/display/NIFI/Release+Notes#ReleaseNotes-Version1.19.0
> > >
> > > The vote will be open for at least 72 hours.
> > > Please download the release candidate and evaluate the necessary items
> > > including checking hashes, signatures, build
> > > from source, and test. Then please vote:
> > >
> > > [ ] +1 Release this package as nifi-1.19.0
> > > [ ] +0 no opinion
> > > [ ] -1 Do not release this package because...
> >
> >

Reply via email to