[GitHub] [nifi-registry] mtien-apache commented on pull request #319: NIFIREG-395 - Implemented the ability to import and export versioned flows through the UI

2021-04-23 Thread GitBox
mtien-apache commented on pull request #319: URL: https://github.com/apache/nifi-registry/pull/319#issuecomment-825935715 @andrewmlim Thanks for reviewing! - The UI changes make sense to me, especially if the names will match what's in the documentation and NiFi. @moranr Do you

[GitHub] [nifi] mattyb149 commented on a change in pull request #5024: NIFI-8320: Fix column mismatch in PutDatabaseRecord

2021-04-23 Thread GitBox
mattyb149 commented on a change in pull request #5024: URL: https://github.com/apache/nifi/pull/5024#discussion_r619497224 ## File path: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutDatabaseRecord.java ##

[jira] [Updated] (NIFI-8470) PutDatabaseRecord truncates strings to 1 character when inserting into PostgreSQL CHAR(n) field

2021-04-23 Thread Sandro Tolaini (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8470?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sandro Tolaini updated NIFI-8470: - Description: When inserting records into a CHAR(5) column in a PostgreSQL table, the value is

[jira] [Updated] (NIFI-8470) PutDatabaseRecord truncates strings to 1 character when inserting into PostgreSQL CHAR(n) field

2021-04-23 Thread Sandro Tolaini (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8470?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sandro Tolaini updated NIFI-8470: - Description: When inserting records into a CHAR(x) column in a PostgreSQL table, the value is

[jira] [Created] (NIFI-8470) PutDatabaseRecord truncates strings to 1 character when inserting into PostgreSQL CHAR(n) field

2021-04-23 Thread Sandro Tolaini (Jira)
Sandro Tolaini created NIFI-8470: Summary: PutDatabaseRecord truncates strings to 1 character when inserting into PostgreSQL CHAR(n) field Key: NIFI-8470 URL: https://issues.apache.org/jira/browse/NIFI-8470

[jira] [Assigned] (NIFI-8469) Change ProcessSession so that commits are asynchronous

2021-04-23 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Payne reassigned NIFI-8469: Assignee: Mark Payne > Change ProcessSession so that commits are asynchronous >

[jira] [Commented] (NIFI-8465) Correct intermittent failures in PasswordBasedEncryptorGroovyTest

2021-04-23 Thread David Handermann (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17331012#comment-17331012 ] David Handermann commented on NIFI-8465: Thanks for tracking down the source of the problem

[jira] [Created] (NIFI-8469) Change ProcessSession so that commits are asynchronous

2021-04-23 Thread Mark Payne (Jira)
Mark Payne created NIFI-8469: Summary: Change ProcessSession so that commits are asynchronous Key: NIFI-8469 URL: https://issues.apache.org/jira/browse/NIFI-8469 Project: Apache NiFi Issue Type:

[jira] [Commented] (NIFI-8465) Correct intermittent failures in PasswordBasedEncryptorGroovyTest

2021-04-23 Thread Paul Grey (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17331010#comment-17331010 ] Paul Grey commented on NIFI-8465: - options: # accept infrequent failures of test case (1/256 odds) #

[jira] [Updated] (NIFI-8466) Offloading the Single Node load balancing target causes ignored backpressure

2021-04-23 Thread Joseph Gresock (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joseph Gresock updated NIFI-8466: - Status: Patch Available (was: In Progress) > Offloading the Single Node load balancing target

[GitHub] [nifi] gresockj opened a new pull request #5025: NIFI-8466: Resolving offload bug with Single Node load balanced queues

2021-04-23 Thread GitBox
gresockj opened a new pull request #5025: URL: https://github.com/apache/nifi/pull/5025 Description of PR Previously, when a cluster with at least 3 nodes offloaded the first node (the one targeted by a Single Node load balanced queue), only the offloaded node updated its

[jira] [Updated] (NIFI-8468) Upgrade Cassandra driver to latest 4.x

2021-04-23 Thread madhavan (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] madhavan updated NIFI-8468: --- Fix Version/s: (was: 1.14.0) > Upgrade Cassandra driver to latest 4.x >

[jira] [Updated] (NIFI-8468) Upgrade Cassandra driver to latest 4.x

2021-04-23 Thread madhavan (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] madhavan updated NIFI-8468: --- Affects Version/s: (was: 1.13.0) 1.14.0 > Upgrade Cassandra driver to latest 4.x >

[jira] [Updated] (NIFI-8468) Upgrade Cassandra driver to latest 4.x

2021-04-23 Thread madhavan (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] madhavan updated NIFI-8468: --- Description: The Cassandra processors use the [3.11.0

[jira] [Commented] (NIFI-8465) Correct intermittent failures in PasswordBasedEncryptorGroovyTest

2021-04-23 Thread Paul Grey (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330993#comment-17330993 ] Paul Grey commented on NIFI-8465: - PasswordBasedEncryptor depends on the contained call to

[jira] [Created] (NIFI-8468) Upgrade Cassandra driver to latest 4.x

2021-04-23 Thread madhavan (Jira)
madhavan created NIFI-8468: -- Summary: Upgrade Cassandra driver to latest 4.x Key: NIFI-8468 URL: https://issues.apache.org/jira/browse/NIFI-8468 Project: Apache NiFi Issue Type: Improvement

[jira] [Commented] (NIFI-8465) Correct intermittent failures in PasswordBasedEncryptorGroovyTest

2021-04-23 Thread Paul Grey (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330988#comment-17330988 ] Paul Grey commented on NIFI-8465: -

[jira] [Comment Edited] (NIFI-8465) Correct intermittent failures in PasswordBasedEncryptorGroovyTest

2021-04-23 Thread Paul Grey (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330985#comment-17330985 ] Paul Grey edited comment on NIFI-8465 at 4/23/21, 6:39 PM: ---

[jira] [Commented] (NIFI-8465) Correct intermittent failures in PasswordBasedEncryptorGroovyTest

2021-04-23 Thread Paul Grey (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330985#comment-17330985 ] Paul Grey commented on NIFI-8465: -

[GitHub] [nifi-registry] andrewmlim commented on pull request #319: NIFIREG-395 - Implemented the ability to import and export versioned flows through the UI

2021-04-23 Thread GitBox
andrewmlim commented on pull request #319: URL: https://github.com/apache/nifi-registry/pull/319#issuecomment-825837500 Related to my last comment, here are the messages in the error dialogs I have found so far: - Non flow definition JSON file: "Cannot create versioned flow snapshot

[GitHub] [nifi] markap14 commented on a change in pull request #5004: NIFI-8433: Added ability to decommission a node in a cluster

2021-04-23 Thread GitBox
markap14 commented on a change in pull request #5004: URL: https://github.com/apache/nifi/pull/5004#discussion_r619407468 ## File path:

[GitHub] [nifi] markap14 commented on pull request #5004: NIFI-8433: Added ability to decommission a node in a cluster

2021-04-23 Thread GitBox
markap14 commented on pull request #5004: URL: https://github.com/apache/nifi/pull/5004#issuecomment-825826144 Thanks @jfrazee. Yeah, I did consider a timeout on some of those wait's. But if it does timeout, i dont know that it's really going to be able to recover. So it could either wait

[GitHub] [nifi-registry] andrewmlim commented on pull request #319: NIFIREG-395 - Implemented the ability to import and export versioned flows through the UI

2021-04-23 Thread GitBox
andrewmlim commented on pull request #319: URL: https://github.com/apache/nifi-registry/pull/319#issuecomment-825819989 Testing how the feature does error handling and noticed the following: When importing new flow, if I select a JSON file that is valid JSON, but not a valid flow

[jira] [Updated] (NIFI-8437) RecordReader 'Infer Schema' for large records with JsonPathReader may throw IOException

2021-04-23 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8437?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Payne updated NIFI-8437: - Summary: RecordReader 'Infer Schema' for large records with JsonPathReader may throw IOException (was:

[jira] [Commented] (NIFI-8437) RecordReader 'Infer Schema' for large records may throw BufferedInputStream error

2021-04-23 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330956#comment-17330956 ] ASF subversion and git services commented on NIFI-8437: --- Commit

[jira] [Resolved] (NIFI-8437) RecordReader 'Infer Schema' for large records may throw BufferedInputStream error

2021-04-23 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8437?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Payne resolved NIFI-8437. -- Fix Version/s: 1.14.0 Resolution: Fixed > RecordReader 'Infer Schema' for large records may

[jira] [Updated] (NIFI-8437) RecordReader 'Infer Schema' for large records with JsonPathReader may throw BufferedInputStream error

2021-04-23 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8437?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Payne updated NIFI-8437: - Summary: RecordReader 'Infer Schema' for large records with JsonPathReader may throw BufferedInputStream

[GitHub] [nifi] markap14 closed pull request #5011: NIFI-8437 RecordReader 'Infer Schema' for large records may throw BufferedInputStream error

2021-04-23 Thread GitBox
markap14 closed pull request #5011: URL: https://github.com/apache/nifi/pull/5011 -- 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. For queries about this service, please

[GitHub] [nifi] markap14 commented on pull request #5011: NIFI-8437 RecordReader 'Infer Schema' for large records may throw BufferedInputStream error

2021-04-23 Thread GitBox
markap14 commented on pull request #5011: URL: https://github.com/apache/nifi/pull/5011#issuecomment-825817860 Thanks @adenes all looks good to me. +1 merged to main -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [nifi-registry] exceptionfactory opened a new pull request #320: NIFIREG-449 Provide AuthenticationManager to avoid password generation

2021-04-23 Thread GitBox
exceptionfactory opened a new pull request #320: URL: https://github.com/apache/nifi-registry/pull/320 Description of PR NIFIREG-449 Updates the Spring Security configuration to expose the `AuthenticationManager` as a bean. This change disables instantiation of the Spring

[GitHub] [nifi-minifi-cpp] arpadboda closed pull request #1046: MINIFICPP-1032 Refactor parse_url

2021-04-23 Thread GitBox
arpadboda closed pull request #1046: URL: https://github.com/apache/nifi-minifi-cpp/pull/1046 -- 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. For queries about this

[GitHub] [nifi-minifi-cpp] arpadboda closed pull request #1058: MINIFICPP-1547 - Change default c2 protocol

2021-04-23 Thread GitBox
arpadboda closed pull request #1058: URL: https://github.com/apache/nifi-minifi-cpp/pull/1058 -- 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. For queries about this

[jira] [Created] (NIFIREG-449) Update Spring Configuration to disable unused password generation

2021-04-23 Thread David Handermann (Jira)
David Handermann created NIFIREG-449: Summary: Update Spring Configuration to disable unused password generation Key: NIFIREG-449 URL: https://issues.apache.org/jira/browse/NIFIREG-449 Project:

[GitHub] [nifi-registry] andrewmlim commented on pull request #319: NIFIREG-395 - Implemented the ability to import and export versioned flows through the UI

2021-04-23 Thread GitBox
andrewmlim commented on pull request #319: URL: https://github.com/apache/nifi-registry/pull/319#issuecomment-825773747 Additional thoughts: - In the "Import New Flow" and "Import New Version" dialogs, change "Data Flow Name" to "Flow Name" for same reasons stated earlier. - In

[jira] [Assigned] (NIFI-8025) Refactor AccessResource

2021-04-23 Thread Nathan Gough (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8025?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nathan Gough reassigned NIFI-8025: -- Assignee: Nathan Gough > Refactor AccessResource > --- > >

[GitHub] [nifi-registry] andrewmlim edited a comment on pull request #319: NIFIREG-395 - Implemented the ability to import and export versioned flows through the UI

2021-04-23 Thread GitBox
andrewmlim edited a comment on pull request #319: URL: https://github.com/apache/nifi-registry/pull/319#issuecomment-825757987 This new functionality is great! It is going to be extremely useful. Here are some thoughts after some UI/UX testing: - I think it would be an

[GitHub] [nifi] jfrazee commented on a change in pull request #5004: NIFI-8433: Added ability to decommission a node in a cluster

2021-04-23 Thread GitBox
jfrazee commented on a change in pull request #5004: URL: https://github.com/apache/nifi/pull/5004#discussion_r619298454 ## File path: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-runtime/src/main/java/org/apache/nifi/BootstrapListener.java ## @@ -202,6 +203,23

[GitHub] [nifi-registry] andrewmlim edited a comment on pull request #319: NIFIREG-395 - Implemented the ability to import and export versioned flows through the UI

2021-04-23 Thread GitBox
andrewmlim edited a comment on pull request #319: URL: https://github.com/apache/nifi-registry/pull/319#issuecomment-825757987 This new functionality is great! It is going to be extremely useful. Here are some thoughts after some UI/UX testing: - I think it would be an

[GitHub] [nifi-registry] andrewmlim commented on pull request #319: NIFIREG-395 - Implemented the ability to import and export versioned flows through the UI

2021-04-23 Thread GitBox
andrewmlim commented on pull request #319: URL: https://github.com/apache/nifi-registry/pull/319#issuecomment-825757987 This new functionality is great! It is going to be extremely useful. Here are some thoughts after some UI/UX testing: - I think it would be an improvement if

[GitHub] [nifi] exceptionfactory commented on a change in pull request #4986: NIFI-8403: Generating Self-signed cert on startup when applicable

2021-04-23 Thread GitBox
exceptionfactory commented on a change in pull request #4986: URL: https://github.com/apache/nifi/pull/4986#discussion_r619326144 ## File path: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/util/SecureNiFiConfigUtil.java ## @@ -0,0 +1,197 @@ +/* + * Licensed to the

[GitHub] [nifi] gresockj commented on a change in pull request #4986: NIFI-8403: Generating Self-signed cert on startup when applicable

2021-04-23 Thread GitBox
gresockj commented on a change in pull request #4986: URL: https://github.com/apache/nifi/pull/4986#discussion_r619319244 ## File path: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/util/SecureNiFiConfigUtil.java ## @@ -0,0 +1,197 @@ +/* + * Licensed to the Apache

[GitHub] [nifi] gresockj commented on a change in pull request #4986: NIFI-8403: Generating Self-signed cert on startup when applicable

2021-04-23 Thread GitBox
gresockj commented on a change in pull request #4986: URL: https://github.com/apache/nifi/pull/4986#discussion_r619319244 ## File path: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/util/SecureNiFiConfigUtil.java ## @@ -0,0 +1,197 @@ +/* + * Licensed to the Apache

[GitHub] [nifi] bbende commented on a change in pull request #4986: NIFI-8403: Generating Self-signed cert on startup when applicable

2021-04-23 Thread GitBox
bbende commented on a change in pull request #4986: URL: https://github.com/apache/nifi/pull/4986#discussion_r619313393 ## File path: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/util/SecureNiFiConfigUtil.java ## @@ -0,0 +1,197 @@ +/* + * Licensed to the Apache

[jira] [Commented] (NIFI-8461) Queue reports items but cannot list them

2021-04-23 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330863#comment-17330863 ] Mark Payne commented on NIFI-8461: -- [~kaagaard] I would try going in your browser to

[GitHub] [nifi-minifi-cpp] szaszm commented on a change in pull request #1060: MINIFI-CPP-1203 - Add linter recommended missing whitespaces

2021-04-23 Thread GitBox
szaszm commented on a change in pull request #1060: URL: https://github.com/apache/nifi-minifi-cpp/pull/1060#discussion_r619285211 ## File path: extensions/sftp/tests/PutSFTPTests.cpp ## @@ -809,7 +809,7 @@ TEST_CASE_METHOD(PutSFTPTestsFixture, "PutSFTP batching does not fail

[GitHub] [nifi-minifi-cpp] szaszm commented on pull request #1060: MINIFI-CPP-1203 - Add linter recommended missing whitespaces

2021-04-23 Thread GitBox
szaszm commented on pull request #1060: URL: https://github.com/apache/nifi-minifi-cpp/pull/1060#issuecomment-825712387 I'm not a fan of space before the colon in range-based for loops. Why does the linter check pass without these changes? Are these rules disabled? -- This is an

[jira] [Assigned] (NIFI-8466) Offloading the Single Node load balancing target causes ignored backpressure

2021-04-23 Thread Joseph Gresock (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joseph Gresock reassigned NIFI-8466: Assignee: Joseph Gresock > Offloading the Single Node load balancing target causes ignored

[GitHub] [nifi-minifi-cpp] hunyadi-dev opened a new pull request #1060: MINIFI-CPP-1203 - Add linter recommended missing whitespaces

2021-04-23 Thread GitBox
hunyadi-dev opened a new pull request #1060: URL: https://github.com/apache/nifi-minifi-cpp/pull/1060 List of errors corrected: ``` 123 Missing space before ( in if( [whitespace/parens] [5] 90 Missing space after , [whitespace/comma] [3] 71 Missing space before {

[jira] [Commented] (NIFI-8467) Improve UX of adding a process group using a flow definition

2021-04-23 Thread Andrew M. Lim (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8467?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330817#comment-17330817 ] Andrew M. Lim commented on NIFI-8467: - Sounds good [~rmoran]! > Improve UX of adding a process group

[jira] [Commented] (NIFI-8467) Improve UX of adding a process group using a flow definition

2021-04-23 Thread Rob Moran (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8467?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330801#comment-17330801 ] Rob Moran commented on NIFI-8467: - Hey [~andrewmlim] you point out very good points here and I agree the

[jira] [Comment Edited] (NIFI-8453) HandleHttpRequest does not receive HTTP request

2021-04-23 Thread Belkacem D (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8453?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17326649#comment-17326649 ] Belkacem D edited comment on NIFI-8453 at 4/23/21, 1:54 PM: For more details,

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #1046: MINIFICPP-1032 Refactor parse_url

2021-04-23 Thread GitBox
fgerlits commented on a change in pull request #1046: URL: https://github.com/apache/nifi-minifi-cpp/pull/1046#discussion_r619233142 ## File path: libminifi/include/RemoteProcessorGroupPort.h ## @@ -147,22 +147,16 @@ class RemoteProcessorGroupPort : public core::Processor {

[GitHub] [nifi-minifi-cpp] szaszm commented on a change in pull request #1042: MINIFICPP-1457 - Implement InputRequirements

2021-04-23 Thread GitBox
szaszm commented on a change in pull request #1042: URL: https://github.com/apache/nifi-minifi-cpp/pull/1042#discussion_r619226036 ## File path: libminifi/include/core/Annotation.h ## @@ -0,0 +1,79 @@ +/** + + * Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1059: MINIFICPP-1510 Register and fix InvokeHTTPTests

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1059: URL: https://github.com/apache/nifi-minifi-cpp/pull/1059#discussion_r619231323 ## File path: extensions/http-curl/tests/CMakeLists.txt ## @@ -40,8 +40,8 @@ FOREACH(testfile ${CURL_UNIT_TESTS})

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1059: MINIFICPP-1510 Register and fix InvokeHTTPTests

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1059: URL: https://github.com/apache/nifi-minifi-cpp/pull/1059#discussion_r619231323 ## File path: extensions/http-curl/tests/CMakeLists.txt ## @@ -40,8 +40,8 @@ FOREACH(testfile ${CURL_UNIT_TESTS})

[GitHub] [nifi-minifi-cpp] szaszm commented on a change in pull request #1042: MINIFICPP-1457 - Implement InputRequirements

2021-04-23 Thread GitBox
szaszm commented on a change in pull request #1042: URL: https://github.com/apache/nifi-minifi-cpp/pull/1042#discussion_r619226036 ## File path: libminifi/include/core/Annotation.h ## @@ -0,0 +1,79 @@ +/** + + * Licensed to the Apache Software Foundation (ASF) under one or

[jira] [Updated] (NIFI-8453) HandleHttpRequest does not receive HTTP request

2021-04-23 Thread Belkacem D (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8453?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Belkacem D updated NIFI-8453: - Description: After starting Nifi, HandleHttpRequest thread remains blocked and does not receive HTTP

[GitHub] [nifi-minifi-cpp] szaszm commented on a change in pull request #1059: MINIFICPP-1510 Register and fix InvokeHTTPTests

2021-04-23 Thread GitBox
szaszm commented on a change in pull request #1059: URL: https://github.com/apache/nifi-minifi-cpp/pull/1059#discussion_r619216945 ## File path: extensions/http-curl/tests/CMakeLists.txt ## @@ -40,8 +40,8 @@ FOREACH(testfile ${CURL_UNIT_TESTS})

[GitHub] [nifi-minifi-cpp] szaszm commented on a change in pull request #1046: MINIFICPP-1032 Refactor parse_url

2021-04-23 Thread GitBox
szaszm commented on a change in pull request #1046: URL: https://github.com/apache/nifi-minifi-cpp/pull/1046#discussion_r619191499 ## File path: libminifi/include/RemoteProcessorGroupPort.h ## @@ -147,22 +147,16 @@ class RemoteProcessorGroupPort : public core::Processor {

[jira] [Commented] (NIFI-8461) Queue reports items but cannot list them

2021-04-23 Thread Kevin Aagaard (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330444#comment-17330444 ] Kevin Aagaard commented on NIFI-8461: - # This processor will route the flow into the REL_FAILURE

[jira] [Updated] (NIFI-8461) Queue reports items but cannot list them

2021-04-23 Thread Kevin Aagaard (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Aagaard updated NIFI-8461: Attachment: image-2021-04-23-07-11-11-125.png > Queue reports items but cannot list them >

[jira] [Updated] (NIFI-8461) Queue reports items but cannot list them

2021-04-23 Thread Kevin Aagaard (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Aagaard updated NIFI-8461: Attachment: (was: image-2021-04-23-07-09-59-737.png) > Queue reports items but cannot list them

[jira] [Updated] (NIFI-8461) Queue reports items but cannot list them

2021-04-23 Thread Kevin Aagaard (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Aagaard updated NIFI-8461: Attachment: (was: image-2021-04-23-07-11-11-125.png) > Queue reports items but cannot list them

[jira] [Updated] (NIFI-8461) Queue reports items but cannot list them

2021-04-23 Thread Kevin Aagaard (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Aagaard updated NIFI-8461: Attachment: image-2021-04-23-07-09-59-737.png > Queue reports items but cannot list them >

[GitHub] [nifi] granthenke commented on pull request #5023: NIFI-8463: Support custom SASL protocol name in PutKudu

2021-04-23 Thread GitBox
granthenke commented on pull request #5023: URL: https://github.com/apache/nifi/pull/5023#issuecomment-825632245  LGTM. We will just need to update the pom with 1.15.0 when Kudu releases. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [nifi-minifi-cpp] hunyadi-dev edited a comment on pull request #1017: MINIFICPP-1515 - Add integration tests testing different flowfile sizes in a simple flow

2021-04-23 Thread GitBox
hunyadi-dev edited a comment on pull request #1017: URL: https://github.com/apache/nifi-minifi-cpp/pull/1017#issuecomment-825620546 The check still shows that **we do not handle the file size 2.1 GiB** properly. Merge **only once it is fixed.** -- This is an automated message from the

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on pull request #1017: MINIFICPP-1515 - Add integration tests testing different flowfile sizes in a simple flow

2021-04-23 Thread GitBox
hunyadi-dev commented on pull request #1017: URL: https://github.com/apache/nifi-minifi-cpp/pull/1017#issuecomment-825620546 The check still shows that **we do not handle the file size 2.1 GiB properly**. Merge once it is fixed. -- This is an automated message from the Apache Git

[jira] [Comment Edited] (NIFI-8466) Offloading the Single Node load balancing target causes ignored backpressure

2021-04-23 Thread Joseph Gresock (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330413#comment-17330413 ] Joseph Gresock edited comment on NIFI-8466 at 4/23/21, 12:20 PM: - One

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on a change in pull request #1017: MINIFICPP-1515 - Add integration tests testing different flowfile sizes in a simple flow

2021-04-23 Thread GitBox
hunyadi-dev commented on a change in pull request #1017: URL: https://github.com/apache/nifi-minifi-cpp/pull/1017#discussion_r619173921 ## File path: docker/test/integration/minifi/validators/SingleFileContentHashValidator.py ## @@ -0,0 +1,43 @@ +import logging +import os +

[jira] [Commented] (NIFI-8466) Offloading the Single Node load balancing target causes ignored backpressure

2021-04-23 Thread Joseph Gresock (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330413#comment-17330413 ] Joseph Gresock commented on NIFI-8466: -- One other observation. When a queue is in this state, it

[jira] [Resolved] (MINIFICPP-1345) Add linter for python scripts

2021-04-23 Thread Gabor Gyimesi (Jira)
[ https://issues.apache.org/jira/browse/MINIFICPP-1345?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gabor Gyimesi resolved MINIFICPP-1345. -- Resolution: Fixed > Add linter for python scripts > -

[jira] [Resolved] (MINIFICPP-1499) Introduce caching to Windows CI builds

2021-04-23 Thread Gabor Gyimesi (Jira)
[ https://issues.apache.org/jira/browse/MINIFICPP-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gabor Gyimesi resolved MINIFICPP-1499. -- Resolution: Fixed > Introduce caching to Windows CI builds >

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1059: MINIFICPP-1510 Register and fix InvokeHTTPTests

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1059: URL: https://github.com/apache/nifi-minifi-cpp/pull/1059#discussion_r619137647 ## File path: extensions/http-curl/tests/CMakeLists.txt ## @@ -68,6 +68,7 @@ message("-- Finished building ${CURL_INT_TEST_COUNT} libcURL

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1032: MINIFICPP-1504: Add Resource consumption data to heartbeats

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1032: URL: https://github.com/apache/nifi-minifi-cpp/pull/1032#discussion_r619122307 ## File path: libminifi/include/utils/ProcessCPUUsageTracker.h ## @@ -0,0 +1,96 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1032: MINIFICPP-1504: Add Resource consumption data to heartbeats

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1032: URL: https://github.com/apache/nifi-minifi-cpp/pull/1032#discussion_r619063072 ## File path: libminifi/include/utils/ProcessCPUUsageTracker.h ## @@ -0,0 +1,96 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1052: MINIFICPP-1244 Support the Initial Start Position property in TailFile

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1052: URL: https://github.com/apache/nifi-minifi-cpp/pull/1052#discussion_r619093502 ## File path: extensions/standard-processors/processors/TailFile.cpp ## @@ -653,29 +688,57 @@ void TailFile::onTrigger(const std::shared_ptr

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1052: MINIFICPP-1244 Support the Initial Start Position property in TailFile

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1052: URL: https://github.com/apache/nifi-minifi-cpp/pull/1052#discussion_r619093317 ## File path: extensions/standard-processors/processors/TailFile.cpp ## @@ -609,6 +640,10 @@ std::vector TailFile::findRotatedFiles(const

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1052: MINIFICPP-1244 Support the Initial Start Position property in TailFile

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1052: URL: https://github.com/apache/nifi-minifi-cpp/pull/1052#discussion_r619093040 ## File path: extensions/standard-processors/processors/TailFile.cpp ## @@ -653,29 +688,57 @@ void TailFile::onTrigger(const std::shared_ptr

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #1059: MINIFICPP-1510 Register and fix InvokeHTTPTests

2021-04-23 Thread GitBox
adamdebreceni commented on a change in pull request #1059: URL: https://github.com/apache/nifi-minifi-cpp/pull/1059#discussion_r619077556 ## File path: extensions/http-curl/tests/CMakeLists.txt ## @@ -68,6 +68,7 @@ message("-- Finished building ${CURL_INT_TEST_COUNT} libcURL

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #1052: MINIFICPP-1244 Support the Initial Start Position property in TailFile

2021-04-23 Thread GitBox
adamdebreceni commented on a change in pull request #1052: URL: https://github.com/apache/nifi-minifi-cpp/pull/1052#discussion_r619061515 ## File path: extensions/standard-processors/processors/TailFile.cpp ## @@ -653,29 +688,57 @@ void TailFile::onTrigger(const

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #1052: MINIFICPP-1244 Support the Initial Start Position property in TailFile

2021-04-23 Thread GitBox
adamdebreceni commented on a change in pull request #1052: URL: https://github.com/apache/nifi-minifi-cpp/pull/1052#discussion_r619055105 ## File path: extensions/standard-processors/processors/TailFile.cpp ## @@ -609,6 +640,10 @@ std::vector TailFile::findRotatedFiles(const

[jira] [Commented] (NIFI-8461) Queue reports items but cannot list them

2021-04-23 Thread Pierre Villard (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17330208#comment-17330208 ] Pierre Villard commented on NIFI-8461: -- What is your ExecuteScript doing? > Queue reports items but

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1054: MINIFICPP-1544 Add Custom Text property to GenerateFlowFile processor

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1054: URL: https://github.com/apache/nifi-minifi-cpp/pull/1054#discussion_r619044246 ## File path: extensions/standard-processors/processors/GenerateFlowFile.cpp ## @@ -61,6 +62,13 @@ core::Property

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1054: MINIFICPP-1544 Add Custom Text property to GenerateFlowFile processor

2021-04-23 Thread GitBox
lordgamez commented on a change in pull request #1054: URL: https://github.com/apache/nifi-minifi-cpp/pull/1054#discussion_r619044081 ## File path: extensions/standard-processors/processors/GenerateFlowFile.cpp ## @@ -110,6 +119,15 @@ void GenerateFlowFile::onSchedule(const

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #1052: MINIFICPP-1244 Support the Initial Start Position property in TailFile

2021-04-23 Thread GitBox
adamdebreceni commented on a change in pull request #1052: URL: https://github.com/apache/nifi-minifi-cpp/pull/1052#discussion_r619043103 ## File path: extensions/standard-processors/processors/TailFile.cpp ## @@ -653,29 +688,57 @@ void TailFile::onTrigger(const

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #1058: MINIFICPP-1547 - Change default c2 protocol

2021-04-23 Thread GitBox
adamdebreceni commented on a change in pull request #1058: URL: https://github.com/apache/nifi-minifi-cpp/pull/1058#discussion_r619015406 ## File path: C2.md ## @@ -63,11 +63,11 @@ be requested via C2 DESCRIBE manifest command. # specify classes for the AST response

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #1046: MINIFICPP-1032 Refactor parse_url

2021-04-23 Thread GitBox
fgerlits commented on a change in pull request #1046: URL: https://github.com/apache/nifi-minifi-cpp/pull/1046#discussion_r619008212 ## File path: libminifi/src/utils/HTTPClient.cpp ## @@ -15,8 +15,43 @@ * See the License for the specific language governing permissions and

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #1046: MINIFICPP-1032 Refactor parse_url

2021-04-23 Thread GitBox
adamdebreceni commented on a change in pull request #1046: URL: https://github.com/apache/nifi-minifi-cpp/pull/1046#discussion_r618987062 ## File path: libminifi/src/utils/HTTPClient.cpp ## @@ -15,8 +15,43 @@ * See the License for the specific language governing permissions