Re: [PR] MINIFICPP-2259 ProcessContext::getProperty should operate on raw poin… [nifi-minifi-cpp]

2024-02-07 Thread via GitHub
szaszm closed pull request #1707: MINIFICPP-2259 ProcessContext::getProperty should operate on raw poin… URL: https://github.com/apache/nifi-minifi-cpp/pull/1707 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] MINIFICPP-2259 ProcessContext::getProperty should operate on raw poin… [nifi-minifi-cpp]

2024-01-10 Thread via GitHub
martinzink commented on code in PR #1707: URL: https://github.com/apache/nifi-minifi-cpp/pull/1707#discussion_r1447266560 ## libminifi/src/core/ProcessSession.cpp: ## @@ -110,36 +109,36 @@ std::shared_ptr ProcessSession::create(const

Re: [PR] MINIFICPP-2259 ProcessContext::getProperty should operate on raw poin… [nifi-minifi-cpp]

2024-01-10 Thread via GitHub
fgerlits commented on code in PR #1707: URL: https://github.com/apache/nifi-minifi-cpp/pull/1707#discussion_r1447129423 ## libminifi/src/core/ProcessSession.cpp: ## @@ -110,36 +109,36 @@ std::shared_ptr ProcessSession::create(const

Re: [PR] MINIFICPP-2259 ProcessContext::getProperty should operate on raw poin… [nifi-minifi-cpp]

2023-12-19 Thread via GitHub
martinzink commented on code in PR #1707: URL: https://github.com/apache/nifi-minifi-cpp/pull/1707#discussion_r1431353285 ## extensions/aws/processors/FetchS3Object.h: ## @@ -46,6 +46,7 @@ class FetchS3Object : public S3Processor { .supportsExpressionLanguage(true)

Re: [PR] MINIFICPP-2259 ProcessContext::getProperty should operate on raw poin… [nifi-minifi-cpp]

2023-12-13 Thread via GitHub
szaszm commented on code in PR #1707: URL: https://github.com/apache/nifi-minifi-cpp/pull/1707#discussion_r1425368129 ## extensions/aws/processors/FetchS3Object.h: ## @@ -46,6 +46,7 @@ class FetchS3Object : public S3Processor { .supportsExpressionLanguage(true)

Re: [PR] MINIFICPP-2259 ProcessContext::getProperty should operate on raw poin… [nifi-minifi-cpp]

2023-12-12 Thread via GitHub
martinzink commented on PR #1707: URL: https://github.com/apache/nifi-minifi-cpp/pull/1707#issuecomment-1851594569 Similarly to https://github.com/apache/nifi-minifi-cpp/pull/1693 I'd ask you to be a bit more lenient with the clang tidy results. I'd rather not refactor the whole codebase

[PR] MINIFICPP-2259 ProcessContext::getProperty should operate on raw poin… [nifi-minifi-cpp]

2023-12-11 Thread via GitHub
martinzink opened a new pull request, #1707: URL: https://github.com/apache/nifi-minifi-cpp/pull/1707 …ters 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