[GitHub] [nifi] VedaKadam opened a new pull request #4504: NIFI-7778: Made corrections in descriptions of padLeft, padRight, plus

2020-08-31 Thread GitBox
VedaKadam opened a new pull request #4504: URL: https://github.com/apache/nifi/pull/4504 Thank you for submitting a contribution to Apache NiFi. Please provide a short description of the PR here: _Add a missing # at the end of descriptions of padLeft, padRight and plus._

[jira] [Updated] (NIFI-7778) Correction in description of padLeft, padRight and plus in EL Guide

2020-08-31 Thread Veda Kadam (Jira)
[ https://issues.apache.org/jira/browse/NIFI-7778?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Veda Kadam updated NIFI-7778: - Description: Add a missing # at the end of descriptions of padLeft, padRight and plus. (was: Added a

[jira] [Created] (NIFI-7778) Correction in description of padLeft, padRight and plus in EL Guide

2020-08-31 Thread Veda Kadam (Jira)
Veda Kadam created NIFI-7778: Summary: Correction in description of padLeft, padRight and plus in EL Guide Key: NIFI-7778 URL: https://issues.apache.org/jira/browse/NIFI-7778 Project: Apache NiFi

[jira] [Created] (NIFI-7777) CompressContent should accept password

2020-08-31 Thread Andy LoPresto (Jira)
Andy LoPresto created NIFI-: --- Summary: CompressContent should accept password Key: NIFI- URL: https://issues.apache.org/jira/browse/NIFI- Project: Apache NiFi Issue Type:

[GitHub] [nifi] thenatog opened a new pull request #4503: NIFI-7767 - Fixed issue with tls-toolkit not adding SANs to generated…

2020-08-31 Thread GitBox
thenatog opened a new pull request #4503: URL: https://github.com/apache/nifi/pull/4503 … certificates. Added tests. NIFI-7767 - Fixed checkstyle. NIFI-7767 - Fixed up TlsCertificateAuthorityTest to include SAN in tests. Thank you for submitting a contribution to Apache

[jira] [Commented] (NIFI-40) Implement Provenance Query Language -- requires refactoring of Prov Repo API / Implementations

2020-08-31 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-40?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187993#comment-17187993 ] Mark Payne commented on NIFI-40: Unlikely. This was one of the first Jiras created for the project and has

[jira] [Updated] (NIFI-7762) Disabled state of Ports is lost when copy-pasted or instanciated from a template

2020-08-31 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-7762?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Payne updated NIFI-7762: - Fix Version/s: 1.13.0 Resolution: Fixed Status: Resolved (was: Patch Available) >

[GitHub] [nifi] markap14 commented on pull request #4491: NIFI-7762: Disabled state of Ports is lost when copy-pasted or instanciated from a template

2020-08-31 Thread GitBox
markap14 commented on pull request #4491: URL: https://github.com/apache/nifi/pull/4491#issuecomment-684004495 Thanks @humpfhumpf all looks good to me! Code makes sense. Was able to verify behavior before and after. +1 merged to main.

[jira] [Commented] (NIFI-7762) Disabled state of Ports is lost when copy-pasted or instanciated from a template

2020-08-31 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/NIFI-7762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187978#comment-17187978 ] ASF subversion and git services commented on NIFI-7762: --- Commit

[GitHub] [nifi] markap14 merged pull request #4491: NIFI-7762: Disabled state of Ports is lost when copy-pasted or instanciated from a template

2020-08-31 Thread GitBox
markap14 merged pull request #4491: URL: https://github.com/apache/nifi/pull/4491 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

[GitHub] [nifi] markap14 commented on pull request #4417: NIFI-7661 Change validation of controller service against it's API to…

2020-08-31 Thread GitBox
markap14 commented on pull request #4417: URL: https://github.com/apache/nifi/pull/4417#issuecomment-683988248 Thanks @bbende looks good. Code is clean, clear & concise. Was able to verify behavior by copying 1.11.4 version of the record-seriation-services-bundle into 1.13.0-SNAPSHOT. Was

[jira] [Commented] (NIFI-7661) Improve validation of controller services against the required API

2020-08-31 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/NIFI-7661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187972#comment-17187972 ] ASF subversion and git services commented on NIFI-7661: --- Commit

[GitHub] [nifi] markap14 merged pull request #4417: NIFI-7661 Change validation of controller service against it's API to…

2020-08-31 Thread GitBox
markap14 merged pull request #4417: URL: https://github.com/apache/nifi/pull/4417 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

[jira] [Updated] (NIFI-7661) Improve validation of controller services against the required API

2020-08-31 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-7661?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Payne updated NIFI-7661: - Fix Version/s: 1.13.0 Resolution: Fixed Status: Resolved (was: Patch Available) >

[GitHub] [nifi] pgyori commented on pull request #4481: NIFI-7624: ListenFTP processor

2020-08-31 Thread GitBox
pgyori commented on pull request #4481: URL: https://github.com/apache/nifi/pull/4481#issuecomment-683974569 @pvillard31 , I committed the changes that enable using SSL, with an SSL context service. This is an automated

[GitHub] [nifi] turcsanyip commented on a change in pull request #4489: NIFI-7740: Add Records Per Transaction and Transactions Per Batch properties to PutHive3Streaming

2020-08-31 Thread GitBox
turcsanyip commented on a change in pull request #4489: URL: https://github.com/apache/nifi/pull/4489#discussion_r480261846 ## File path: nifi-nar-bundles/nifi-hive-bundle/nifi-hive3-processors/src/main/java/org/apache/hive/streaming/RecordsEOFException.java ## @@ -0,0 +1,28

[GitHub] [nifi] xasx opened a new pull request #4502: NIFI-7754 Support HTTP Proxy for JettyWebSocketClient

2020-08-31 Thread GitBox
xasx opened a new pull request #4502: URL: https://github.com/apache/nifi/pull/4502 Description of PR Enables supporting an HTTP Proxy for JettyWebSocketClient; fixes bug NIFI-7754. In order to streamline the review of the contribution we ask you to ensure the

[GitHub] [nifi] xasx closed pull request #4485: NIFI-7754: Support HTTP Proxy for JettyWebSocketClient

2020-08-31 Thread GitBox
xasx closed pull request #4485: URL: https://github.com/apache/nifi/pull/4485 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

[GitHub] [nifi-minifi-cpp] szaszm commented on a change in pull request #857: MINIFICPP-1309 - RAII based resourceClaim management

2020-08-31 Thread GitBox
szaszm commented on a change in pull request #857: URL: https://github.com/apache/nifi-minifi-cpp/pull/857#discussion_r480027949 ## File path: libminifi/include/ResourceClaim.h ## @@ -45,8 +45,10 @@ extern std::string default_directory_path; extern void

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
hunyadi-dev commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480192949 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the Apache

[GitHub] [nifi-minifi-cpp] szaszm commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
szaszm commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480189463 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the Apache

[GitHub] [nifi-minifi-cpp] lordgamez opened a new pull request #888: MINIFICPP-1321 Do not override MergeContent output attributes

2020-08-31 Thread GitBox
lordgamez opened a new pull request #888: URL: https://github.com/apache/nifi-minifi-cpp/pull/888 This commit handles the case when flowfiles processed by MergeContent could have common attributes, with the same value (like mime type) which after the attribute merge would override the

[GitHub] [nifi-minifi-cpp] szaszm commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
szaszm commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480185712 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the Apache

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
adamdebreceni commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480170258 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
adamdebreceni commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480165898 ## File path: libminifi/src/core/yaml/CheckRequiredField.cpp ## @@ -0,0 +1,55 @@ +/** + * + * Licensed to the Apache Software Foundation

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
hunyadi-dev commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480164518 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the Apache

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
adamdebreceni commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480151533 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
adamdebreceni commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480146776 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
hunyadi-dev commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480147201 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the Apache

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
hunyadi-dev commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480147201 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the Apache

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
adamdebreceni commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480146776 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
hunyadi-dev commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480145721 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the Apache

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on pull request #884: MINIFICPP-1203 - Correct linter reported missing spaces around comments

2020-08-31 Thread GitBox
hunyadi-dev commented on pull request #884: URL: https://github.com/apache/nifi-minifi-cpp/pull/884#issuecomment-683792142 I agree that some of the comments are redundant, but I considered cleaning them up out-of-scope for this PR. I can add a commit cleaning them up though.

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #885: MINIFICPP-1344 - Correct rest endpoint for buckets on updateFromPayload

2020-08-31 Thread GitBox
fgerlits commented on a change in pull request #885: URL: https://github.com/apache/nifi-minifi-cpp/pull/885#discussion_r480137667 ## File path: libminifi/src/core/FlowConfiguration.cpp ## @@ -67,33 +67,24 @@ std::shared_ptr FlowConfiguration::createProvenanceReportTask()

[GitHub] [nifi] simonbence closed pull request #4398: NIFI-7255 Cleaning up obsolete nodes when connecting to cluster

2020-08-31 Thread GitBox
simonbence closed pull request #4398: URL: https://github.com/apache/nifi/pull/4398 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

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
adamdebreceni commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480144761 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the

[GitHub] [nifi] simonbence commented on pull request #4398: NIFI-7255 Cleaning up obsolete nodes when connecting to cluster

2020-08-31 Thread GitBox
simonbence commented on pull request #4398: URL: https://github.com/apache/nifi/pull/4398#issuecomment-683791213 As this effects only older versions, changing the current main does not seems to be a satisfying solution. This review will be abandoned, a different approach is taken.

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #885: MINIFICPP-1344 - Correct rest endpoint for buckets on updateFromPayload

2020-08-31 Thread GitBox
fgerlits commented on a change in pull request #885: URL: https://github.com/apache/nifi-minifi-cpp/pull/885#discussion_r480131679 ## File path: libminifi/src/core/FlowConfiguration.cpp ## @@ -67,33 +67,24 @@ std::shared_ptr FlowConfiguration::createProvenanceReportTask()

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #885: MINIFICPP-1344 - Correct rest endpoint for buckets on updateFromPayload

2020-08-31 Thread GitBox
fgerlits commented on a change in pull request #885: URL: https://github.com/apache/nifi-minifi-cpp/pull/885#discussion_r480137667 ## File path: libminifi/src/core/FlowConfiguration.cpp ## @@ -67,33 +67,24 @@ std::shared_ptr FlowConfiguration::createProvenanceReportTask()

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on a change in pull request #874: Minificpp 1325 - Refactor and test YAML connection parsing

2020-08-31 Thread GitBox
adamdebreceni commented on a change in pull request #874: URL: https://github.com/apache/nifi-minifi-cpp/pull/874#discussion_r480136156 ## File path: extensions/standard-processors/tests/unit/YamlConnectionParserTest.cpp ## @@ -0,0 +1,200 @@ +/** + * + * Licensed to the

[GitHub] [nifi] adarmiento commented on pull request #4441: NIFI 7525 - Create a nullValue() Subjectless Functions and support null type on Expression Language

2020-08-31 Thread GitBox
adarmiento commented on pull request #4441: URL: https://github.com/apache/nifi/pull/4441#issuecomment-683779159 Hello @Fabryprog, Whatever the outcome of the discussion with the community, I would undersline that you are not actually interested in the null value itself (if I

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #884: MINIFICPP-1203 - Correct linter reported missing spaces around comments

2020-08-31 Thread GitBox
fgerlits commented on a change in pull request #884: URL: https://github.com/apache/nifi-minifi-cpp/pull/884#discussion_r480120751 ## File path: extensions/http-curl/tests/HTTPHandlers.h ## @@ -428,7 +428,7 @@ class HeartbeatHandler : public ServerAwareHandler { void

[jira] [Resolved] (NIFI-7544) ReplaceText processor seems not thread-safe?

2020-08-31 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-7544?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Payne resolved NIFI-7544. -- Fix Version/s: 1.12.0 Resolution: Duplicate > ReplaceText processor seems not thread-safe? >

[GitHub] [nifi] mgaido91 commented on pull request #4441: NIFI 7525 - Create a nullValue() Subjectless Functions and support null type on Expression Language

2020-08-31 Thread GitBox
mgaido91 commented on pull request #4441: URL: https://github.com/apache/nifi/pull/4441#issuecomment-683769823 I think you can maybe send an email in the community list to check whether this is a feature many people need and in case of a favorable result we can go ahead with the PR.

[GitHub] [nifi] Fabryprog commented on pull request #4441: NIFI 7525 - Create a nullValue() Subjectless Functions and support null type on Expression Language

2020-08-31 Thread GitBox
Fabryprog commented on pull request #4441: URL: https://github.com/apache/nifi/pull/4441#issuecomment-683768322 > I don't see the usefulness of such a method and I am not sure it is worth the big code change, sorry Hello Marco! Thanks for your time! In the past I've been use

[GitHub] [nifi-minifi-cpp] szaszm commented on pull request #857: MINIFICPP-1309 - RAII based resourceClaim management

2020-08-31 Thread GitBox
szaszm commented on pull request #857: URL: https://github.com/apache/nifi-minifi-cpp/pull/857#issuecomment-683763463 :eyes: This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [nifi] mgaido91 commented on pull request #4441: NIFI 7525 - Create a nullValue() Subjectless Functions and support null type on Expression Language

2020-08-31 Thread GitBox
mgaido91 commented on pull request #4441: URL: https://github.com/apache/nifi/pull/4441#issuecomment-683761763 I don't see the usefulness of such a method and I am not sure it is worth the big code change, sorry This is an

[GitHub] [nifi-minifi-cpp] fgerlits commented on pull request #886: MINIFICPP-1323 Encrypt sensitive properties

2020-08-31 Thread GitBox
fgerlits commented on pull request #886: URL: https://github.com/apache/nifi-minifi-cpp/pull/886#issuecomment-683755809 > cool stuff, as I understand it only aims to protect the user when they (accidentally or intentionally) expose the `minifi.properties` file Yes, that is true.

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #878: MINIFICPP-1104 Add SSL docker tests for PublishKafka and automate them in CI

2020-08-31 Thread GitBox
lordgamez commented on a change in pull request #878: URL: https://github.com/apache/nifi-minifi-cpp/pull/878#discussion_r480091219 ## File path: docker/test/integration/minifi/__init__.py ## @@ -413,7 +426,7 @@ def __init__(self, self.controller_services =

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

2020-08-31 Thread Gabor Gyimesi (Jira)
Gabor Gyimesi created MINIFICPP-1345: Summary: Add linter for scripts Key: MINIFICPP-1345 URL: https://issues.apache.org/jira/browse/MINIFICPP-1345 Project: Apache NiFi MiNiFi C++ Issue

[jira] [Updated] (NIFI-7762) Disabled state of Ports is lost when copy-pasted or instanciated from a template

2020-08-31 Thread humpfhumpf (Jira)
[ https://issues.apache.org/jira/browse/NIFI-7762?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] humpfhumpf updated NIFI-7762: - Status: Patch Available (was: Open) > Disabled state of Ports is lost when copy-pasted or instanciated

[jira] [Updated] (NIFI-7771) Infinite loop on WebUI when node stopped in cluster

2020-08-31 Thread humpfhumpf (Jira)
[ https://issues.apache.org/jira/browse/NIFI-7771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] humpfhumpf updated NIFI-7771: - Status: Patch Available (was: Open) > Infinite loop on WebUI when node stopped in cluster >

[GitHub] [nifi-minifi-cpp] adamdebreceni commented on pull request #886: MINIFICPP-1323 Encrypt sensitive properties

2020-08-31 Thread GitBox
adamdebreceni commented on pull request #886: URL: https://github.com/apache/nifi-minifi-cpp/pull/886#issuecomment-683731126 cool stuff, as I understand it only aims to protect the user when they (accidentally or intentionally) expose the `minifi.properties` file

[GitHub] [nifi-minifi-cpp] adamdebreceni opened a new pull request #887: MINIFICPP-1340 - Batch persist content changes

2020-08-31 Thread GitBox
adamdebreceni opened a new pull request #887: URL: https://github.com/apache/nifi-minifi-cpp/pull/887 Thank you for submitting a contribution to Apache NiFi - MiNiFi C++. In order to streamline the review of the contribution we ask you to ensure the following steps have been

[GitHub] [nifi-minifi-cpp] lordgamez commented on pull request #884: MINIFICPP-1203 - Correct linter reported missing spaces around comments

2020-08-31 Thread GitBox
lordgamez commented on pull request #884: URL: https://github.com/apache/nifi-minifi-cpp/pull/884#issuecomment-683664280 > > > > Is there a plan/jira ticket for also running these checks (with possibly an additionally shellcheck for shell files and flake8/pylint check for python files) in

[GitHub] [nifi-minifi-cpp] fgerlits opened a new pull request #886: MINIFICPP-1323 Encrypt sensitive properties

2020-08-31 Thread GitBox
fgerlits opened a new pull request #886: URL: https://github.com/apache/nifi-minifi-cpp/pull/886 https://issues.apache.org/jira/browse/MINIFICPP-1323 Encrypt sensitive properties in the `minifi.properties` file. The change has two parts: 1. implement a new command-line tool

[GitHub] [nifi-minifi-cpp] hunyadi-dev edited a comment on pull request #884: MINIFICPP-1203 - Correct linter reported missing spaces around comments

2020-08-31 Thread GitBox
hunyadi-dev edited a comment on pull request #884: URL: https://github.com/apache/nifi-minifi-cpp/pull/884#issuecomment-683658116 > > > Is there a plan/jira ticket for also running these checks (with possibly an additionally shellcheck for shell files and flake8/pylint check for python

[GitHub] [nifi-minifi-cpp] hunyadi-dev edited a comment on pull request #884: MINIFICPP-1203 - Correct linter reported missing spaces around comments

2020-08-31 Thread GitBox
hunyadi-dev edited a comment on pull request #884: URL: https://github.com/apache/nifi-minifi-cpp/pull/884#issuecomment-683658116 > > > Is there a plan/jira ticket for also running these checks (with possibly an additionally shellcheck for shell files and flake8/pylint check for python

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on pull request #884: MINIFICPP-1203 - Correct linter reported missing spaces around comments

2020-08-31 Thread GitBox
hunyadi-dev commented on pull request #884: URL: https://github.com/apache/nifi-minifi-cpp/pull/884#issuecomment-683658116 > > > Is there a plan/jira ticket for also running these checks (with possibly an additionally shellcheck for shell files and flake8/pylint check for python files) in

[GitHub] [nifi-minifi-cpp] lordgamez commented on pull request #884: MINIFICPP-1203 - Correct linter reported missing spaces around comments

2020-08-31 Thread GitBox
lordgamez commented on pull request #884: URL: https://github.com/apache/nifi-minifi-cpp/pull/884#issuecomment-683651016 > > Is there a plan/jira ticket for also running these checks (with possibly an additionally shellcheck for shell files and flake8/pylint check for python files) in the

[GitHub] [nifi-minifi-cpp] hunyadi-dev opened a new pull request #885: MINIFICPP-1344 - Correct rest endpoint for buckets on updateFromPayload

2020-08-31 Thread GitBox
hunyadi-dev opened a new pull request #885: URL: https://github.com/apache/nifi-minifi-cpp/pull/885 I tried to predict what kind of URL the function meant to parse and changed the implementation based on that. The only important change in logic is that now the function looks for "buckets"

[jira] [Created] (MINIFICPP-1344) Investigate and fix FlowConfiguration payload URLs

2020-08-31 Thread Adam Hunyadi (Jira)
Adam Hunyadi created MINIFICPP-1344: --- Summary: Investigate and fix FlowConfiguration payload URLs Key: MINIFICPP-1344 URL: https://issues.apache.org/jira/browse/MINIFICPP-1344 Project: Apache NiFi

[GitHub] [nifi] ecerulm commented on pull request #4465: NIFI-7721 nifi.sh to return exit code

2020-08-31 Thread GitBox
ecerulm commented on pull request #4465: URL: https://github.com/apache/nifi/pull/4465#issuecomment-683601046 I believe that maybe we should follow the same conventions as `service` * start: 0 only if nifi is started or already running. * stop: 0 if nifi is stopped at the end of

[jira] [Resolved] (MINIFICPP-1339) Revive rocksdb database when the disk has space

2020-08-31 Thread Adam Debreceni (Jira)
[ https://issues.apache.org/jira/browse/MINIFICPP-1339?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Adam Debreceni resolved MINIFICPP-1339. --- Resolution: Fixed > Revive rocksdb database when the disk has space >

[GitHub] [nifi-minifi-cpp] hunyadi-dev commented on pull request #884: MINIFICPP-1203 - Correct linter reported missing spaces around comments

2020-08-31 Thread GitBox
hunyadi-dev commented on pull request #884: URL: https://github.com/apache/nifi-minifi-cpp/pull/884#issuecomment-683593188 > Is there a plan/jira ticket for also running these checks (with possibly an additionally shellcheck for shell files and flake8/pylint check for python files) in the