[GitHub] nifi issue #2828: NIFI-5195 Added clustering to the docker image

2018-07-02 Thread dkhwangbo
Github user dkhwangbo commented on the issue:

https://github.com/apache/nifi/pull/2828
  
@MikeThomsen Hi. Any change is in this PR to make secured-cluster?


---


[GitHub] nifi issue #2696: NIFI-5188: DruidTranquilityController does not fully suppo...

2018-06-13 Thread dkhwangbo
Github user dkhwangbo commented on the issue:

https://github.com/apache/nifi/pull/2696
  
@mattyb149 Manual modification is only way to update LICENSE files? Any 
tools or scripts to do with?


---


[GitHub] nifi issue #2696: NIFI-5188: DruidTranquilityController does not fully suppo...

2018-05-25 Thread dkhwangbo
Github user dkhwangbo commented on the issue:

https://github.com/apache/nifi/pull/2696
  
@joewitt @pvillard31 @mattyb149 Any ideas?


---


[GitHub] nifi issue #2696: NIFI-5188: DruidTranquilityController does not fully suppo...

2018-05-21 Thread dkhwangbo
Github user dkhwangbo commented on the issue:

https://github.com/apache/nifi/pull/2696
  
@mattyb149 Hi. Thanks for your comment. 
I already use this patch builded with Druid 0.9.2 for a week and no 
abnormal operation is detected. But I think Using Druid 0.9.1 is appropriate 
way because latest version of tranquility is builded against Druid 0.9.1(refer: 
[tranquility 0.8.2 release 
note](https://github.com/druid-io/tranquility/releases/tag/v0.8.2)).
I rollback this change. Please review.


---


[GitHub] nifi issue #2696: NIFI-5188: DruidTranquilityController does not fully suppo...

2018-05-15 Thread dkhwangbo
Github user dkhwangbo commented on the issue:

https://github.com/apache/nifi/pull/2696
  
@joewitt @pvillard31 @mattyb149 Any ideas?


---


[GitHub] nifi pull request #2696: NIFI-5188: DruidTranquilityController does not full...

2018-05-14 Thread dkhwangbo
GitHub user dkhwangbo opened a pull request:

https://github.com/apache/nifi/pull/2696

NIFI-5188: DruidTranquilityController does not fully support Druid 
aggregator

Thank you for submitting a contribution to Apache NiFi.

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

### For all changes:
- [x] Is there a JIRA ticket associated with this PR? Is it referenced 
 in the commit message?

- [x] Does your PR title start with NIFI- where  is the JIRA number 
you are trying to resolve? Pay particular attention to the hyphen "-" character.

- [x] Has your PR been rebased against the latest commit within the target 
branch (typically master)?

- [x] Is your initial contribution a single, squashed commit?

### For code changes:
- [x] Have you ensured that the full suite of tests is executed via mvn 
-Pcontrib-check clean install at the root nifi folder?
- [ ] Have you written or updated unit tests to verify your changes?
  - I think verify this PR is enough with unit test already used.
- [x] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)? 
- [ ] If applicable, have you updated the LICENSE file, including the main 
LICENSE file under nifi-assembly?
- [ ] If applicable, have you updated the NOTICE file, including the main 
NOTICE file found under nifi-assembly?
- [ ] If adding new Properties, have you added .displayName in addition to 
.name (programmatic access) for each of the new properties?

### For documentation related changes:
- [ ] Have you ensured that format looks appropriate for the output in 
which it is rendered?

### Note:
Please ensure that once the PR is submitted, you check travis-ci for build 
issues and submit an update to your PR as soon as possible.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/dkhwangbo/nifi NIFI-5188

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/2696.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2696


commit b1080be5469b6e1011b560070c77201441689415
Author: Dongkyu Hwangbo <hwangbodk@...>
Date:   2018-05-14T07:33:49Z

NIFI-5188: DruidTranquilityController does not fully support Druid 
aggregator




---