[GitHub] nifi issue #2512: NIFI-4936 pushed down version declarations to lowest appro...

2018-03-07 Thread jtstorck 
Github user jtstorck commented on the issue:

https://github.com/apache/nifi/pull/2512
  
+1 regarding the HDFS component POM changes.  After building with this PR, 
I tested Put/List/DeleteHDFS processors with TDE paths successfully.


---

[GitHub] nifi issue #2512: NIFI-4936 pushed down version declarations to lowest appro...

2018-03-09 Thread bbende 
Github user bbende commented on the issue:

https://github.com/apache/nifi/pull/2512
  
+1 for the HBase processors, did a quick test of Put/Get against secure and 
unsecure HBase and everything worked fine, bundled dependencies look good


---

[GitHub] nifi issue #2512: NIFI-4936 pushed down version declarations to lowest appro...

2018-03-09 Thread joewitt 
Github user joewitt commented on the issue:

https://github.com/apache/nifi/pull/2512
  
just force pushed a rebased update since recent changes on master have 
altered poms


---

[GitHub] nifi issue #2512: NIFI-4936 pushed down version declarations to lowest appro...

2018-03-09 Thread mcgilman 
Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/2512
  
Will review from a framework perspective...


---

[GitHub] nifi issue #2512: NIFI-4936 pushed down version declarations to lowest appro...

2018-03-09 Thread mcgilman 
Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/2512
  
Thanks @joewitt! I've merged this to master. I've verified various 
framework behavior including standalone vs cluster, ldap authentication, custom 
ui's and data viewers. All looks good.


---