+1 binding On Mon, Nov 28, 2022 at 5:35 AM Peter Turcsanyi <[email protected]> wrote:
> +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 <[email protected]> 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 > > <[email protected]> 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 <[email protected]> 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... > > > > > > >
