[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/nifi-minifi-cpp/pull/83 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r117070877 --- Diff: libminifi/test/resources/cn.ckey.pem --- @@ -0,0 +1,31 @@ +Bag Attributes --- End diff -- Added a reference to that wiki e

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r117022477 --- Diff: libminifi/src/SchedulingAgent.cpp --- @@ -39,6 +40,36 @@ bool SchedulingAgent::hasWorkToDo(std::shared_ptr processor) { return false

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r117021448 --- Diff: libminifi/src/SchedulingAgent.cpp --- @@ -39,6 +40,36 @@ bool SchedulingAgent::hasWorkToDo(std::shared_ptr processor) { return fa

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116983866 --- Diff: libminifi/test/Test.cpp --- @@ -1,6 +1,4 @@ /** - * @file MiNiFiMain.cpp --- End diff -- Ah yeah this one is per ano

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116982739 --- Diff: libminifi/test/resources/cn.ckey.pem --- @@ -0,0 +1,31 @@ +Bag Attributes --- End diff -- Might not be a bad idea, but not ne

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116968835 --- Diff: libminifi/include/processors/ListenHTTP.h --- @@ -115,6 +114,9 @@ class ListenHTTP : public core::Processor { std::unique_ptr _handl

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116968718 --- Diff: libminifi/test/resources/cn.ckey.pem --- @@ -0,0 +1,31 @@ +Bag Attributes --- End diff -- yes. i just used the toolkit to

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116968276 --- Diff: libminifi/test/resources/nifi-cert.pem --- @@ -0,0 +1,20 @@ +-BEGIN CERTIFICATE- --- End diff -- yes --- If your

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116968270 --- Diff: libminifi/test/resources/cn.crt.pem --- @@ -0,0 +1,25 @@ +Bag Attributes --- End diff -- yes --- If your project is set

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116968301 --- Diff: libminifi/test/unit/ClassLoaderTests.cpp --- @@ -0,0 +1,48 @@ +/** + * + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-17 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116968249 --- Diff: libminifi/test/resources/TestControllerServices.yml --- @@ -0,0 +1,62 @@ +Flow Controller: --- End diff -- Yah. that was a

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116895119 --- Diff: libminifi/include/processors/TailFile.h --- @@ -89,6 +89,8 @@ class TailFile : public core::Processor { }; +REGISTER_RESO

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116894473 --- Diff: libminifi/src/core/ClassLoader.cpp --- @@ -0,0 +1,70 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116895225 --- Diff: libminifi/include/processors/TailFile.h --- @@ -73,8 +74,7 @@ class TailFile : public core::Processor { std::string _stateFile;

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116892423 --- Diff: libminifi/test/unit/MockClasses.h --- @@ -0,0 +1,139 @@ +/** + * + * Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116894602 --- Diff: libminifi/src/SchedulingAgent.cpp --- @@ -39,6 +40,36 @@ bool SchedulingAgent::hasWorkToDo(std::shared_ptr processor) { return false

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116892947 --- Diff: libminifi/test/resources/cn.ckey.pem --- @@ -0,0 +1,31 @@ +Bag Attributes --- End diff -- Was this something you generated?

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116892828 --- Diff: libminifi/test/resources/cn.crt.pem --- @@ -0,0 +1,25 @@ +Bag Attributes --- End diff -- Was this something you generated?

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116893614 --- Diff: libminifi/test/Test.cpp --- @@ -1,6 +1,4 @@ /** - * @file MiNiFiMain.cpp --- End diff -- Should this file just be nixed

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116893221 --- Diff: libminifi/test/resources/TestControllerServices.yml --- @@ -0,0 +1,62 @@ +Flow Controller: --- End diff -- Can't leave commen

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116892776 --- Diff: libminifi/test/resources/nifi-cert.pem --- @@ -0,0 +1,20 @@ +-BEGIN CERTIFICATE- --- End diff -- Was this something y

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116895271 --- Diff: libminifi/include/processors/ListenHTTP.h --- @@ -115,6 +114,9 @@ class ListenHTTP : public core::Processor { std::unique_ptr _handler;

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116895039 --- Diff: libminifi/include/utils/StringUtils.h --- @@ -18,6 +18,7 @@ #define LIBMINIFI_INCLUDE_IO_STRINGUTILS_H_ #include +#includ

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-05-16 Thread apiri
Github user apiri commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/83#discussion_r116892700 --- Diff: libminifi/test/unit/ClassLoaderTests.cpp --- @@ -0,0 +1,48 @@ +/** + * + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] nifi-minifi-cpp pull request #83: MINIFI-226: Add controller services capabi...

2017-04-25 Thread phrocker
GitHub user phrocker opened a pull request: https://github.com/apache/nifi-minifi-cpp/pull/83 MINIFI-226: Add controller services capabilities along with unit tests MINIFI-253 : Add basic Process Loader capability Thank you for submitting a contribution to Apache NiFi - MiNi