[GitHub] nifi-minifi-cpp pull request #460: MINIFICPP-479: Add processor property des...

2018-12-12 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/460#discussion_r241199779 --- Diff: libminifi/include/core/state/Value.h --- @@ -74,94 +157,215 @@ class BoolValue : public Value { explicit BoolValue(bool value)

[GitHub] nifi-minifi-cpp pull request #460: MINIFICPP-479: Add processor property des...

2018-12-12 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/460#discussion_r241193264 --- Diff: libminifi/include/core/PropertyValidation.h --- @@ -0,0 +1,347 @@ +/** + * + * Licensed to the Apache Software Foundation

[GitHub] nifi-minifi-cpp pull request #460: MINIFICPP-479: Add processor property des...

2018-12-12 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/460#discussion_r241192961 --- Diff: libminifi/include/core/state/Value.h --- @@ -74,94 +157,215 @@ class BoolValue : public Value { explicit BoolValue(bool value)

[GitHub] nifi-minifi-cpp pull request #460: MINIFICPP-479: Add processor property des...

2018-12-12 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/460#discussion_r241051202 --- Diff: libminifi/src/processors/LogAttribute.cpp --- @@ -38,13 +38,23 @@ namespace apache { namespace nifi { namespace minifi {

[GitHub] nifi-minifi-cpp pull request #460: MINIFICPP-479: Add processor property des...

2018-12-12 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/460#discussion_r241048908 --- Diff: libminifi/include/core/state/Value.h --- @@ -74,94 +157,215 @@ class BoolValue : public Value { explicit BoolValue(bool value)

[GitHub] nifi-minifi-cpp pull request #460: MINIFICPP-479: Add processor property des...

2018-12-12 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/460#discussion_r241052650 --- Diff: libminifi/include/core/PropertyValidation.h --- @@ -0,0 +1,347 @@ +/** + * + * Licensed to the Apache Software Foundation

[GitHub] nifi-minifi-cpp pull request #460: MINIFICPP-479: Add processor property des...

2018-12-12 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/460#discussion_r241048231 --- Diff: libminifi/include/core/state/Value.h --- @@ -74,94 +157,215 @@ class BoolValue : public Value { explicit BoolValue(bool value)

[GitHub] nifi-minifi-cpp pull request #460: MINIFICPP-479: Add processor property des...

2018-12-11 Thread phrocker
GitHub user phrocker opened a pull request: https://github.com/apache/nifi-minifi-cpp/pull/460 MINIFICPP-479: Add processor property descriptor updates. WIP Thank you for submitting a contribution to Apache NiFi - MiNiFi C++. In order to streamline the review of the