[PR] NIFI-13009 Update documentation and ui message to better capture changes made to disconnected node [nifi]

2025-09-26 Thread via GitHub
NissimShiman opened a new pull request, #10354: URL: https://github.com/apache/nifi/pull/10354 # Summary [NIFI-13009](https://issues.apache.org/jira/browse/NIFI-13009) # Tracking Please complete the following tracking st

[PR] NIFI-15017 Add start/stop C2 command for processors in MiNiFi [nifi]

2025-09-26 Thread via GitHub
Mihhai opened a new pull request, #10353: URL: https://github.com/apache/nifi/pull/10353 # Summary [NIFI-15017](https://issues.apache.org/jira/browse/NIFI-15017) # Tracking Please complete the following tracking steps pr

Re: [PR] NIFI-15014 allow advanced UpdateAttribute Rules to be searched even i… [nifi]

2025-09-26 Thread via GitHub
pvillard31 commented on PR #10345: URL: https://github.com/apache/nifi/pull/10345#issuecomment-806218 Yeah fair point, given this is typescript, I assumed that the UI folks are more familiar with this code -- This is an automated message from the Apache Git Service. To respond to the

[PR] MINIFICPP-2638 Fix package source and gitignore lists [nifi-minifi-cpp]

2025-09-26 Thread via GitHub
lordgamez opened a new pull request, #2037: URL: https://github.com/apache/nifi-minifi-cpp/pull/2037 - Remove build_description.cpp from package source ignore list - Add asset test generated files to ignore list - Add CMakePresets.json and CMakeUserPresets.json to ignore list htt

Re: [PR] MINIFICPP-2558 Revamp site to site communication [nifi-minifi-cpp]

2025-09-26 Thread via GitHub
lordgamez commented on code in PR #1966: URL: https://github.com/apache/nifi-minifi-cpp/pull/1966#discussion_r2381549857 ## libminifi/include/sitetosite/SiteToSiteClient.h: ## @@ -19,246 +19,180 @@ #pragma once #include -#include #include #include #include #include

Re: [PR] MINIFICPP-2638 Fix package source and gitignore lists [nifi-minifi-cpp]

2025-09-26 Thread via GitHub
fgerlits closed pull request #2037: MINIFICPP-2638 Fix package source and gitignore lists URL: https://github.com/apache/nifi-minifi-cpp/pull/2037 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] MINIFICPP-2232 Remove option to load SSL context from flow config [nifi-minifi-cpp]

2025-09-26 Thread via GitHub
fgerlits commented on code in PR #1986: URL: https://github.com/apache/nifi-minifi-cpp/pull/1986#discussion_r2381448847 ## libminifi/test/unit/GeneralUtilsTest.cpp: ## @@ -27,55 +27,55 @@ namespace org::apache::nifi::minifi::test { // intdiv_ceil -static_assert(0 == utils::i

Re: [PR] NIFI-15001 - Fix PutElasticsearchRecord with recursive reads [nifi]

2025-09-25 Thread via GitHub
dan-s1 commented on code in PR #10334: URL: https://github.com/apache/nifi/pull/10334#discussion_r2379424428 ## nifi-extension-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-restapi-processors/src/main/java/org/apache/nifi/processors/elasticsearch/PutElasticsearchRecord.java:

[PR] NIFI-15005 route not found [nifi]

2025-09-25 Thread via GitHub
scottyaslan opened a new pull request, #10350: URL: https://github.com/apache/nifi/pull/10350 # Summary [NIFI-15005](https://issues.apache.org/jira/browse/NIFI-15005) https://github.com/user-attachments/assets/c1099db3-67cb-4b5c-bf0e-f32486b1bf48"; /> # Tracking Pleas

Re: [PR] NIFI-15004 nifi registry about dialog [nifi]

2025-09-25 Thread via GitHub
sardell commented on code in PR #10347: URL: https://github.com/apache/nifi/pull/10347#discussion_r2380292986 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/service/about.service.ts: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Foundation (ASF) unde

Re: [PR] NIFI-15004 nifi registry about dialog [nifi]

2025-09-25 Thread via GitHub
sardell commented on code in PR #10347: URL: https://github.com/apache/nifi/pull/10347#discussion_r2380262296 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/service/about.service.ts: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Foundation (ASF) unde

Re: [PR] NIFI-15004 nifi registry about dialog [nifi]

2025-09-25 Thread via GitHub
sardell commented on code in PR #10347: URL: https://github.com/apache/nifi/pull/10347#discussion_r2380262296 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/service/about.service.ts: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Foundation (ASF) unde

Re: [PR] NIFI-15004 nifi registry about dialog [nifi]

2025-09-25 Thread via GitHub
sardell commented on PR #10347: URL: https://github.com/apache/nifi/pull/10347#issuecomment-3335401254 Reviewing now... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To u

Re: [PR] NIFI-14320 buckets listing and management, (manage bucket policies is… [nifi]

2025-09-25 Thread via GitHub
sardell commented on code in PR #10346: URL: https://github.com/apache/nifi/pull/10346#discussion_r2379890984 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/service/buckets.service.ts: ## @@ -37,6 +39,61 @@ export class BucketsService { // }); //

Re: [PR] MINIFICPP-2232 Remove option to load SSL context from flow config [nifi-minifi-cpp]

2025-09-25 Thread via GitHub
szaszm commented on code in PR #1986: URL: https://github.com/apache/nifi-minifi-cpp/pull/1986#discussion_r2379880066 ## libminifi/test/unit/GeneralUtilsTest.cpp: ## Review Comment: What changed that made the `minifi::` prefix necessary? If it's not necessary, I prefer not

Re: [PR] MINIFICPP-2232 Remove option to load SSL context from flow config [nifi-minifi-cpp]

2025-09-25 Thread via GitHub
szaszm commented on code in PR #1986: URL: https://github.com/apache/nifi-minifi-cpp/pull/1986#discussion_r2379877605 ## libminifi/test/unit/GeneralUtilsTest.cpp: ## @@ -27,55 +27,55 @@ namespace org::apache::nifi::minifi::test { // intdiv_ceil -static_assert(0 == utils::int

Re: [PR] MINIFICPP-2232 Remove option to load SSL context from flow config [nifi-minifi-cpp]

2025-09-25 Thread via GitHub
fgerlits commented on code in PR #1986: URL: https://github.com/apache/nifi-minifi-cpp/pull/1986#discussion_r2379810881 ## libminifi/test/unit/GeneralUtilsTest.cpp: ## @@ -27,55 +27,55 @@ namespace org::apache::nifi::minifi::test { // intdiv_ceil -static_assert(0 == utils::i

Re: [PR] NIFI-15012 - FlowDifferenceFilters - fix handling of createControllerService() [nifi]

2025-09-25 Thread via GitHub
bbende merged PR #10342: URL: https://github.com/apache/nifi/pull/10342 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.o

Re: [PR] NIFI-14320 buckets listing and management, (manage bucket policies is… [nifi]

2025-09-25 Thread via GitHub
sardell commented on code in PR #10346: URL: https://github.com/apache/nifi/pull/10346#discussion_r2379713723 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/pages/buckets/feature/ui/bucket-table-filter/bucket-table-filter.component.ts: ## @@ -0,0 +1,103 @@ +/* + *

[PR] NIFI-15021 - Support custom form parameters in StandardOauth2AccessTokenProvider [nifi]

2025-09-25 Thread via GitHub
pvillard31 opened a new pull request, #10352: URL: https://github.com/apache/nifi/pull/10352 # Summary NIFI-15021 - Support custom form parameters in `StandardOauth2AccessTokenProvider` Similar to what we have in `JWTBearerOAuth2AccessTokenProvider`, we should support dynamic

Re: [PR] NIFI-14320 buckets listing and management, (manage bucket policies is… [nifi]

2025-09-25 Thread via GitHub
sardell commented on code in PR #10346: URL: https://github.com/apache/nifi/pull/10346#discussion_r2379685316 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/pages/buckets/feature/ui/bucket-table-filter/bucket-table-filter.component.ts: ## @@ -0,0 +1,103 @@ +/* + *

Re: [PR] NIFI-14320 buckets listing and management, (manage bucket policies is… [nifi]

2025-09-25 Thread via GitHub
sardell commented on code in PR #10346: URL: https://github.com/apache/nifi/pull/10346#discussion_r2379662043 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/app-routing.module.ts: ## @@ -28,13 +28,16 @@ const routes: Routes = [ path: 'explorer',

Re: [PR] MINIFICPP-2558 Revamp site to site communication [nifi-minifi-cpp]

2025-09-25 Thread via GitHub
szaszm commented on code in PR #1966: URL: https://github.com/apache/nifi-minifi-cpp/pull/1966#discussion_r2351674789 ## libminifi/include/sitetosite/SiteToSiteClient.h: ## @@ -19,246 +19,180 @@ #pragma once #include -#include #include #include #include #include #

Re: [PR] NIFI-15001 - Fix PutElasticsearchRecord with recursive reads [nifi]

2025-09-25 Thread via GitHub
pvillard31 commented on code in PR #10334: URL: https://github.com/apache/nifi/pull/10334#discussion_r2379579949 ## nifi-extension-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-restapi-processors/src/main/java/org/apache/nifi/processors/elasticsearch/PutElasticsearchRecord.java:

Re: [PR] NIFI-15010 - FlowDifferenceFilters - better handle property renaming [nifi]

2025-09-25 Thread via GitHub
bbende merged PR #10340: URL: https://github.com/apache/nifi/pull/10340 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.o

[PR] NIFI-14990 Aligned those processors and controller services whose property names do not match the human-friendly display names in extended bundle nifi-elasticsearch-bundle. [nifi]

2025-09-25 Thread via GitHub
dan-s1 opened a new pull request, #10351: URL: https://github.com/apache/nifi/pull/10351 # Summary [NIFI-14990](https://issues.apache.org/jira/browse/NIFI-14990) # Tracking Please complete the following tracking steps pr

Re: [PR] NIFI-14320 buckets listing and management, (manage bucket policies is… [nifi]

2025-09-25 Thread via GitHub
sardell commented on PR #10346: URL: https://github.com/apache/nifi/pull/10346#issuecomment-3334675801 Reviewing now... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To u

Re: [PR] MINIFICPP-2614 Add more authentication options to Azure processors [nifi-minifi-cpp]

2025-09-25 Thread via GitHub
lordgamez commented on code in PR #2016: URL: https://github.com/apache/nifi-minifi-cpp/pull/2016#discussion_r2379485860 ## extensions/azure/controllerservices/AzureStorageCredentialsService.cpp: ## @@ -29,6 +29,10 @@ void AzureStorageCredentialsService::initialize() { } voi

Re: [PR] MINIFICPP-2614 Add more authentication options to Azure processors [nifi-minifi-cpp]

2025-09-25 Thread via GitHub
fgerlits commented on code in PR #2016: URL: https://github.com/apache/nifi-minifi-cpp/pull/2016#discussion_r2378951626 ## extensions/azure/controllerservices/AzureStorageCredentialsService.cpp: ## @@ -29,6 +29,10 @@ void AzureStorageCredentialsService::initialize() { } void

Re: [PR] NIFI-15001 - Fix PutElasticsearchRecord with recursive reads [nifi]

2025-09-25 Thread via GitHub
pvillard31 commented on code in PR #10334: URL: https://github.com/apache/nifi/pull/10334#discussion_r2379363843 ## nifi-extension-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-restapi-processors/src/main/java/org/apache/nifi/processors/elasticsearch/PutElasticsearchRecord.java:

Re: [PR] NIFI-15001 - Fix PutElasticsearchRecord with recursive reads [nifi]

2025-09-25 Thread via GitHub
dan-s1 commented on code in PR #10334: URL: https://github.com/apache/nifi/pull/10334#discussion_r2379276797 ## nifi-extension-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-restapi-processors/src/main/java/org/apache/nifi/processors/elasticsearch/PutElasticsearchRecord.java:

Re: [PR] NIFI-15014 allow advanced UpdateAttribute Rules to be searched even i… [nifi]

2025-09-25 Thread via GitHub
markobean commented on PR #10345: URL: https://github.com/apache/nifi/pull/10345#issuecomment-780219 @pvillard31 This issue is not directly related to the UI; I'm not sure the label is appropriate. While the search functionality is accessed through the UI, the only update is in the back

[PR] NIFI-15015 - Bump Commons Lang3 to 3.19.0, H2 to 2.4.240, Datafaker to 2.5.1, and others [nifi]

2025-09-25 Thread via GitHub
pvillard31 opened a new pull request, #10349: URL: https://github.com/apache/nifi/pull/10349 # Summary NIFI-15015 - Bump Commons Lang3 to 3.19.0, H2 to 2.4.240, Datafaker to 2.5.1, and others - Spotbugs Maven Plugin from 4.9.5.0 to 4.9.6.0 - https://github.com/spotbugs/spotbug

Re: [PR] NIFI-15009 Codemirror json highlighting [nifi]

2025-09-24 Thread via GitHub
rfellows merged PR #10339: URL: https://github.com/apache/nifi/pull/10339 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache

[PR] NIFI-15007 add link to documentation in global menu [nifi]

2025-09-24 Thread via GitHub
scottyaslan opened a new pull request, #10348: URL: https://github.com/apache/nifi/pull/10348 # Summary [NIFI-15007](https://issues.apache.org/jira/browse/NIFI-15007) link to nifi registry documentation. https://github.com/user-attachments/assets/5fa2861d-dacc-430d-9a6d-6ef3a8c

[PR] NIFI-15004 nifi registry about dialog [nifi]

2025-09-24 Thread via GitHub
scottyaslan opened a new pull request, #10347: URL: https://github.com/apache/nifi/pull/10347 # Summary [NIFI-15004](https://issues.apache.org/jira/browse/NIFI-15004) https://github.com/user-attachments/assets/c349ee0a-3486-4686-b763-421b7284eb0a"; /> # Tracking

[PR] NIFI-15012 - FlowDifferenceFilters - fix handling of createControllerService() [nifi]

2025-09-24 Thread via GitHub
pvillard31 opened a new pull request, #10342: URL: https://github.com/apache/nifi/pull/10342 # Summary NIFI-15012 - FlowDifferenceFilters - fix handling of createControllerService() In [NIFI-14995](https://issues.apache.org/jira/browse/NIFI-14995), we tried to properly handle

Re: [PR] MINIFICPP-2637 Upgrade libxml2 to v2.15.0 [nifi-minifi-cpp]

2025-09-24 Thread via GitHub
szaszm commented on code in PR #2036: URL: https://github.com/apache/nifi-minifi-cpp/pull/2036#discussion_r2376155639 ## cmake/LibXml2.cmake: ## @@ -0,0 +1,50 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOT

Re: [PR] MINIFICPP-2637 Upgrade libxml2 to v2.15.0 [nifi-minifi-cpp]

2025-09-24 Thread via GitHub
szaszm commented on code in PR #2036: URL: https://github.com/apache/nifi-minifi-cpp/pull/2036#discussion_r2376153364 ## cmake/LibXml2.cmake: ## Review Comment: Now that it's changed to FetchContent, we should stop referencing libxml2 from the main CMakeLists.txt, and only

Re: [PR] MINIFICPP-2637 Upgrade libxml2 to v2.15.0 [nifi-minifi-cpp]

2025-09-24 Thread via GitHub
szaszm commented on code in PR #2036: URL: https://github.com/apache/nifi-minifi-cpp/pull/2036#discussion_r2376153364 ## cmake/LibXml2.cmake: ## Review Comment: Now that it's changed to FetchContent, we should stop referencing libxml2 from the main CMakeLists.txt, and only

Re: [PR] MINIFICPP-2637 Upgrade libxml2 to v2.15.0 [nifi-minifi-cpp]

2025-09-24 Thread via GitHub
fgerlits commented on code in PR #2036: URL: https://github.com/apache/nifi-minifi-cpp/pull/2036#discussion_r2376086865 ## cmake/AzureSdkCpp.cmake: ## @@ -55,3 +55,8 @@ FetchContent_Declare(asdkext ) FetchContent_MakeAvailable(asdkext) + +if (NOT WIN32) +add_dependencies

[PR] MINIFICPP-2637 Upgrade libxml2 to v2.15.0 [nifi-minifi-cpp]

2025-09-24 Thread via GitHub
lordgamez opened a new pull request, #2036: URL: https://github.com/apache/nifi-minifi-cpp/pull/2036 https://issues.apache.org/jira/browse/MINIFICPP-2637 --- Thank you for submitting a contribution to Apache NiFi - MiNiFi C++. In order to streamline th

Re: [PR] MINIFICPP-2635 Handle dynamic properties correctly in python processors [nifi-minifi-cpp]

2025-09-24 Thread via GitHub
martinzink closed pull request #2034: MINIFICPP-2635 Handle dynamic properties correctly in python processors URL: https://github.com/apache/nifi-minifi-cpp/pull/2034 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] NIFI-14953 Validate inlined protobuf schemas [nifi]

2025-09-23 Thread via GitHub
lkuchars commented on code in PR #10291: URL: https://github.com/apache/nifi/pull/10291#discussion_r2373544878 ## nifi-extension-bundles/nifi-protobuf-bundle/nifi-protobuf-services/src/main/java/org/apache/nifi/services/protobuf/StandardProtobufReader.java: ## @@ -170,6 +174,84

Re: [PR] NIFI-14953 Validate inlined protobuf schemas [nifi]

2025-09-23 Thread via GitHub
lkuchars commented on code in PR #10291: URL: https://github.com/apache/nifi/pull/10291#discussion_r2373543832 ## nifi-extension-bundles/nifi-protobuf-bundle/nifi-protobuf-services/src/test/java/org/apache/nifi/services/protobuf/TestStandardProtobufReaderPropertyValidation.java: ###

Re: [PR] NIFI-14953 Validate inlined protobuf schemas [nifi]

2025-09-23 Thread via GitHub
lkuchars commented on code in PR #10291: URL: https://github.com/apache/nifi/pull/10291#discussion_r2373543562 ## nifi-extension-bundles/nifi-protobuf-bundle/nifi-protobuf-services/src/main/java/org/apache/nifi/services/protobuf/ProtobufSchemaCompiler.java: ## @@ -129,7 +129,8 @

Re: [PR] NIFI-14953 Validate inlined protobuf schemas [nifi]

2025-09-23 Thread via GitHub
lkuchars closed pull request #10291: NIFI-14953 Validate inlined protobuf schemas URL: https://github.com/apache/nifi/pull/10291 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[PR] NIFI-15008 Allow affected components to be reloaded if necessary duri… [nifi]

2025-09-23 Thread via GitHub
bbende opened a new pull request, #10338: URL: https://github.com/apache/nifi/pull/10338 …ng parameter context update # Summary [NIFI-15008](https://issues.apache.org/jira/browse/NIFI-15008) Allow affected components to

Re: [PR] NIFI-14319: Create initial project structure for new NiFi Registry UI [nifi]

2025-09-23 Thread via GitHub
scottyaslan merged PR #9855: URL: https://github.com/apache/nifi/pull/9855 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apach

Re: [PR] MINIFICPP-2635 Handle dynamic properties correctly in python processors [nifi-minifi-cpp]

2025-09-23 Thread via GitHub
fgerlits commented on code in PR #2034: URL: https://github.com/apache/nifi-minifi-cpp/pull/2034#discussion_r2371923043 ## PROCESSORS.md: ## @@ -722,7 +722,7 @@ In the list below, the names of required properties appear in bold. Any other pr ### Description -Executes a scr

Re: [PR] MINIFICPP-2629 Clean up minifi-api includes [nifi-minifi-cpp]

2025-09-23 Thread via GitHub
szaszm commented on code in PR #2029: URL: https://github.com/apache/nifi-minifi-cpp/pull/2029#discussion_r2368628272 ## extensions/systemd/ConsumeJournald.cpp: ## @@ -78,7 +78,7 @@ void ConsumeJournald::onSchedule(core::ProcessContext& context, core::ProcessSes return pro

Re: [PR] MINIFICPP-2629 Clean up minifi-api includes [nifi-minifi-cpp]

2025-09-23 Thread via GitHub
szaszm commented on code in PR #2029: URL: https://github.com/apache/nifi-minifi-cpp/pull/2029#discussion_r2371999120 ## extensions/sftp/processors/ListSFTP.cpp: ## @@ -690,7 +690,7 @@ void ListSFTP::listByTrackingEntities( time_t now = time(nullptr); uint64_t min_timest

Re: [PR] MINIFICPP-2565 Add compression support for site to site communication [nifi-minifi-cpp]

2025-09-23 Thread via GitHub
fgerlits commented on code in PR #1974: URL: https://github.com/apache/nifi-minifi-cpp/pull/1974#discussion_r2371870968 ## SITE_TO_SITE.md: ## Review Comment: this file needs to be added to `packaging/rpm/expected-rpm-contents.in` to fix the Rocky (docker) build -- Thi

Re: [PR] NIFI-14959 fix consistent hash function for partition-by-attribute [nifi]

2025-09-23 Thread via GitHub
pvillard31 closed pull request #10336: NIFI-14959 fix consistent hash function for partition-by-attribute URL: https://github.com/apache/nifi/pull/10336 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] NIFI-14319: Create initial project structure for new NiFi Registry UI [nifi]

2025-09-22 Thread via GitHub
scottyaslan commented on code in PR #9855: URL: https://github.com/apache/nifi/pull/9855#discussion_r2370420338 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/service/error-helper.service.ts: ## @@ -0,0 +1,97 @@ +/* + * Licensed to the Apache Software Foundation (

[PR] NIFI-15000 Remove AzureGraphUserGroupProvider [nifi]

2025-09-22 Thread via GitHub
exceptionfactory opened a new pull request, #10335: URL: https://github.com/apache/nifi/pull/10335 # Summary [NIFI-15000](https://issues.apache.org/jira/browse/NIFI-15000) Removes the `AzureGraphUserGroupProvider` implementation of the framework User Group Provider. The Azure

Re: [PR] NIFI-14384 Adds Force Path Style support to S3 Bundle Persistence Pro… [nifi]

2025-09-22 Thread via GitHub
exceptionfactory commented on code in PR #10331: URL: https://github.com/apache/nifi/pull/10331#discussion_r2370151822 ## nifi-registry/nifi-registry-extensions/nifi-registry-aws/nifi-registry-aws-extensions/src/test/java/org/apache/nifi/registry/aws/S3BundlePersistenceProviderIT.java:

Re: [PR] NIFI-14989 Aligned those processors and controller services whose property names do not match the human-friendly display names in extended bundles nifi-box-bundle, nifi-cdc, nifi-cipher-bundl

2025-09-22 Thread via GitHub
exceptionfactory merged PR #10328: URL: https://github.com/apache/nifi/pull/10328 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nif

Re: [PR] MINIFICPP-2565 Add compression support for site to site communication [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
lordgamez commented on code in PR #1974: URL: https://github.com/apache/nifi-minifi-cpp/pull/1974#discussion_r2368673344 ## libminifi/include/sitetosite/CompressionConsts.h: ## @@ -0,0 +1,28 @@ +/** + * + * Licensed to the Apache Software Foundation (ASF) under one or more + * c

Re: [PR] NIFI-14995 - Improve Flow Differences Filter to account for renameProperty and createControllerService [nifi]

2025-09-22 Thread via GitHub
pvillard31 commented on PR #10327: URL: https://github.com/apache/nifi/pull/10327#issuecomment-3321160596 Thanks for the review @exceptionfactory - I pushed a commit to address your comments -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] NIFI-14995 - Improve Flow Differences Filter to account for renameProperty and createControllerService [nifi]

2025-09-22 Thread via GitHub
exceptionfactory commented on code in PR #10327: URL: https://github.com/apache/nifi/pull/10327#discussion_r2369816063 ## nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/util/FlowDifferenceFilters.java: ## @@ -547,4 +577,302 @@ privat

Re: [PR] MINIFICPP-2634 Turn on building stable extensions by default [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
martinzink closed pull request #2033: MINIFICPP-2634 Turn on building stable extensions by default URL: https://github.com/apache/nifi-minifi-cpp/pull/2033 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] NIFI-14998 - Bump Log4J2 to 2.25.2, Guava to 33.5.0-jre, Mockito to 5.20.0, and others [nifi]

2025-09-22 Thread via GitHub
exceptionfactory merged PR #10333: URL: https://github.com/apache/nifi/pull/10333 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nif

Re: [PR] MINIFICPP-2636 update ExecutePythonProcessor description [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
martinzink closed pull request #2035: MINIFICPP-2636 update ExecutePythonProcessor description URL: https://github.com/apache/nifi-minifi-cpp/pull/2035 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go t

Re: [PR] MINIFICPP-2633 Improve logging for python initialization when no dependencies are installed [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
martinzink closed pull request #2032: MINIFICPP-2633 Improve logging for python initialization when no dependencies are installed URL: https://github.com/apache/nifi-minifi-cpp/pull/2032 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] MINIFICPP-2632 Make RPM creation in rockylinux docker build optional [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
martinzink closed pull request #2031: MINIFICPP-2632 Make RPM creation in rockylinux docker build optional URL: https://github.com/apache/nifi-minifi-cpp/pull/2031 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[PR] NIFI-15001 - Fix PutElasticsearchRecord with recursive reads [nifi]

2025-09-22 Thread via GitHub
pvillard31 opened a new pull request, #10334: URL: https://github.com/apache/nifi/pull/10334 # Summary NIFI-15001 - Fix `PutElasticsearchRecord` with recursive reads When `PutElasticsearchRecord` processed FlowFiles with batch sizes smaller than the record count, it re-read the

Re: [PR] NIFI-14991 Add Process Group Attributes to Context of Component Log [nifi]

2025-09-22 Thread via GitHub
pvillard31 closed pull request #10330: NIFI-14991 Add Process Group Attributes to Context of Component Log URL: https://github.com/apache/nifi/pull/10330 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] NIFI-14991 Add Process Group Attributes to Context of Component Log [nifi]

2025-09-22 Thread via GitHub
exceptionfactory commented on PR #10330: URL: https://github.com/apache/nifi/pull/10330#issuecomment-3319845533 Thanks for the feedback @pvillard31! I pushed an update adding a new section on Mapped Diagnostic Context to the Admin Guide. -- This is an automated message from the Apache Git

Re: [PR] NIFI-14998 - Bump Log4J2 to 2.25.2, Guava to 33.5.0-jre, Mockito to 5.20.0, and others [nifi]

2025-09-22 Thread via GitHub
pvillard31 commented on PR #10333: URL: https://github.com/apache/nifi/pull/10333#issuecomment-3319834937 Done, thanks for the feedback @exceptionfactory -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

Re: [PR] MINIFICPP-2629 Clean up minifi-api includes [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
szaszm commented on code in PR #2029: URL: https://github.com/apache/nifi-minifi-cpp/pull/2029#discussion_r2368523396 ## extensions/sftp/processors/ListSFTP.cpp: ## @@ -690,7 +690,7 @@ void ListSFTP::listByTrackingEntities( time_t now = time(nullptr); uint64_t min_timest

Re: [PR] docs: fix small typos & formatting in README [nifi]

2025-09-22 Thread via GitHub
exceptionfactory closed pull request #10332: docs: fix small typos & formatting in README URL: https://github.com/apache/nifi/pull/10332 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] MINIFICPP-2614 Add more authentication options to Azure processors [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
lordgamez commented on code in PR #2016: URL: https://github.com/apache/nifi-minifi-cpp/pull/2016#discussion_r2368801960 ## cmake/AzureSdkCpp.cmake: ## @@ -55,3 +55,5 @@ FetchContent_Declare(asdkext ) FetchContent_MakeAvailable(asdkext) + +add_dependencies(azure-storage-comm

Re: [PR] MINIFICPP-2614 Add more authentication options to Azure processors [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
lordgamez commented on code in PR #2016: URL: https://github.com/apache/nifi-minifi-cpp/pull/2016#discussion_r2368789434 ## extensions/azure/tests/ListAzureDataLakeStorageTests.cpp: ## @@ -43,7 +43,7 @@ class ListAzureDataLakeStorageTestsFixture { auto mock_data_lake_storag

Re: [PR] MINIFICPP-2565 Add compression support for site to site communication [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
lordgamez commented on code in PR #1974: URL: https://github.com/apache/nifi-minifi-cpp/pull/1974#discussion_r2368583835 ## core-framework/include/io/CRCStream.h: ## @@ -66,6 +66,10 @@ class CRCStreamBase : public virtual StreamImpl { crc_ = crc32(0L, Z_NULL, 0); } +

Re: [PR] MINIFICPP-2636 update ExecutePythonProcessor description [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
lordgamez commented on PR #2035: URL: https://github.com/apache/nifi-minifi-cpp/pull/2035#issuecomment-3319153087 On the question regarding the API, I think we should keep it for now. We still had users recently using the old API and I would not want to break those processors as it is not t

Re: [PR] MINIFICPP-2614 Add more authentication options to Azure processors [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
szaszm commented on code in PR #2016: URL: https://github.com/apache/nifi-minifi-cpp/pull/2016#discussion_r2352309301 ## cmake/AzureSdkCpp.cmake: ## @@ -55,3 +55,5 @@ FetchContent_Declare(asdkext ) FetchContent_MakeAvailable(asdkext) + +add_dependencies(azure-storage-common

[PR] MINIFICPP-2636 update ExecutePythonProcessor description [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
szaszm opened a new pull request, #2035: URL: https://github.com/apache/nifi-minifi-cpp/pull/2035 - onTrigger accepts session, not property - justifying basic behavior with efficiency seems weird, removed - remove the sentence about concurrency, since the processor is single threaded

[PR] MINIFICPP-2635 Handle dynamic properties correctly in python processors [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
lordgamez opened a new pull request, #2034: URL: https://github.com/apache/nifi-minifi-cpp/pull/2034 - Fix dynamic property support flag setting in python processors - Only support dynamic properties in NiFi style python processors if `getDynamicPropertyDescriptor` is defined - Fix `yi

Re: [PR] MINIFICPP-2614 Add more authentication options to Azure processors [nifi-minifi-cpp]

2025-09-22 Thread via GitHub
adamdebreceni commented on code in PR #2016: URL: https://github.com/apache/nifi-minifi-cpp/pull/2016#discussion_r2368006469 ## extensions/azure/tests/ListAzureDataLakeStorageTests.cpp: ## @@ -43,7 +43,7 @@ class ListAzureDataLakeStorageTestsFixture { auto mock_data_lake_st

Re: [PR] NIFI-10752: Implement GetCouchbase and PutCouchbase processors with SDK3 [nifi]

2025-09-22 Thread via GitHub
mark-bathori commented on code in PR #10031: URL: https://github.com/apache/nifi/pull/10031#discussion_r2367929046 ## nifi-extension-bundles/nifi-couchbase-bundle/nifi-couchbase-services-api/src/main/java/org/apache/nifi/services/couchbase/CouchbaseClient.java: ## @@ -0,0 +1,31

Re: [PR] NIFI-10752: Implement GetCouchbase and PutCouchbase processors with SDK3 [nifi]

2025-09-22 Thread via GitHub
mark-bathori commented on code in PR #10031: URL: https://github.com/apache/nifi/pull/10031#discussion_r2367351304 ## nifi-extension-bundles/nifi-couchbase-bundle/nifi-couchbase-services-api/src/main/java/org/apache/nifi/services/couchbase/CouchbaseClient.java: ## @@ -0,0 +1,31

[PR] NIFI-14998 - Bump Log4J2 to 2.25.2, Guava to 33.5.0-jre, Mockito to 5.20.0, and others [nifi]

2025-09-22 Thread via GitHub
pvillard31 opened a new pull request, #10333: URL: https://github.com/apache/nifi/pull/10333 # Summary NIFI-14998 - Bump Log4J2 to 2.25.2, Guava to 33.5.0-jre, Mockito to 5.20.0, and others - Jetbrains Annotations from 24.1.0 to 26.0.2-1 - https://github.com/JetBrains/java-ann

Re: [PR] NIFI-14727 Create new ConsumeKinesis Processor [nifi]

2025-09-22 Thread via GitHub
awelless commented on code in PR #10077: URL: https://github.com/apache/nifi/pull/10077#discussion_r2367134960 ## nifi-extension-bundles/nifi-aws-bundle/nifi-aws-kinesis/src/main/java/org/apache/nifi/processors/aws/kinesis/MemoryBoundRecordBuffer.java: ## @@ -0,0 +1,673 @@ +/* +

[PR] docs: fix small typos & formatting in README [nifi]

2025-09-21 Thread via GitHub
Username46786 opened a new pull request, #10332: URL: https://github.com/apache/nifi/pull/10332 “SAML 2” → “SAML 2.0” (matches admin guide terminology) Corrected grep example: nifi-app*.log and quoted pattern Added missing comma in license sentence Docs-only changes. -- This is a

Re: [PR] NIFI-14900 - Adding support for dropStateKeySupported Stateful flag [nifi]

2025-09-20 Thread via GitHub
exceptionfactory commented on code in PR #10238: URL: https://github.com/apache/nifi/pull/10238#discussion_r2337201860 ## nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/controller/state/StandardStateManager.java: ## @@ -24,21 +24,34

Re: [PR] NIFI-14892 Aligned those processors and controller services whose property names do not match the human-friendly display names in extended bundles nifi-airtable-bundle, nifi-amqp-bundle, nifi

2025-09-20 Thread via GitHub
exceptionfactory merged PR #10292: URL: https://github.com/apache/nifi/pull/10292 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nif

Re: [PR] NIFI-14927 - Expose Assume Role (ARN/Session Name) for AWS_MSK_IAM in AmazonMSKConnectionService [nifi]

2025-09-20 Thread via GitHub
exceptionfactory merged PR #10278: URL: https://github.com/apache/nifi/pull/10278 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nif

Re: [PR] NIFI-14988: Fixing issue with default values for dynamic properties. [nifi]

2025-09-20 Thread via GitHub
rfellows commented on PR #10320: URL: https://github.com/apache/nifi/pull/10320#issuecomment-3307706373 reviewing... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsu

Re: [PR] NIFI-14910 NIFI-14911 NiFi - Upgrade Jetty to 12.1.1 and move to Jakarta EE11 [nifi]

2025-09-20 Thread via GitHub
pvillard31 commented on PR #10249: URL: https://github.com/apache/nifi/pull/10249#issuecomment-3299385576 OK... so while using Jetty 12.1.1 works for NiFi Registry (I was able to properly build it, run it, and use it with NiFi), the integration tests are failing because of the way Spring Bo

Re: [PR] NIFI-10752: Implement GetCouchbase and PutCouchbase processors with SDK3 [nifi]

2025-09-20 Thread via GitHub
turcsanyip commented on code in PR #10031: URL: https://github.com/apache/nifi/pull/10031#discussion_r2352389740 ## nifi-extension-bundles/nifi-couchbase-bundle/nifi-couchbase-processors/src/main/java/org/apache/nifi/processors/couchbase/GetCouchbase.java: ## @@ -0,0 +1,98 @@ +/

Re: [PR] NIFI-14945 - Bump Jetty to 12.0.26, JOSE JWT to 10.5, AWS SDK v2 to 2.33.4, and others [nifi]

2025-09-20 Thread via GitHub
exceptionfactory merged PR #10282: URL: https://github.com/apache/nifi/pull/10282 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nif

Re: [PR] Bump vite from 5.4.19 to 5.4.20 in /nifi-frontend/src/main/frontend [nifi]

2025-09-20 Thread via GitHub
mcgilman commented on PR #10290: URL: https://github.com/apache/nifi/pull/10290#issuecomment-3285264046 Thanks @exceptionfactory! We'll be bumping to Angular 20 soon, hopefully next week. I suspect that upgrade will include this update as well. -- This is an automated message from the Apa

Re: [PR] NIFI-14960 - Bump Spring to 6.2.11, Jetty to 12.0.27, Protobuf to 4.32.1, and others [nifi]

2025-09-20 Thread via GitHub
exceptionfactory merged PR #10297: URL: https://github.com/apache/nifi/pull/10297 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nif

Re: [PR] NIFI-14970 Replaced deprecated isEmpty method of net.sf.saxon.s9api.XdmValue with isEmptySequence(). [nifi]

2025-09-20 Thread via GitHub
exceptionfactory merged PR #10305: URL: https://github.com/apache/nifi/pull/10305 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nif

Re: [PR] MINIFICPP-2305 Fix various C2 integration test issues [nifi-minifi-cpp]

2025-09-20 Thread via GitHub
lordgamez commented on code in PR #2022: URL: https://github.com/apache/nifi-minifi-cpp/pull/2022#discussion_r2352081674 ## libminifi/test/libtest/integration/IntegrationBase.cpp: ## Review Comment: Good point, added a comment in https://github.com/apache/nifi-minifi-cpp/p

Re: [PR] MINIFICPP-2626 Update RPM package description [nifi-minifi-cpp]

2025-09-20 Thread via GitHub
martinzink commented on code in PR #2026: URL: https://github.com/apache/nifi-minifi-cpp/pull/2026#discussion_r2336420866 ## CMakeLists.txt: ## @@ -673,7 +673,28 @@ else() if (MINIFI_PACKAGING_TYPE STREQUAL "RPM") set(CPACK_GENERATOR "RPM") set(CPACK_BINAR

Re: [PR] NIFI-14950: Allowing individual state entries to be removed when the … [nifi]

2025-09-20 Thread via GitHub
rfellows merged PR #10287: URL: https://github.com/apache/nifi/pull/10287 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache

Re: [PR] NIFI-14319: Create initial project structure for new NiFi Registry UI [nifi]

2025-09-20 Thread via GitHub
sardell commented on code in PR #9855: URL: https://github.com/apache/nifi/pull/9855#discussion_r2345491422 ## nifi-frontend/src/main/frontend/apps/nifi-registry/src/app/service/droplets.service.ts: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

Re: [PR] NIFI-14992 - Remove deprecated PMD rule UselessOperationOnImmutable [nifi]

2025-09-20 Thread via GitHub
exceptionfactory merged PR #10323: URL: https://github.com/apache/nifi/pull/10323 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nif

  1   2   3   4   5   6   7   8   9   10   >