[GitHub] nifi issue #2712: NIFI-5197 Fixed a bunch of EL scope bugs.

2018-05-17 Thread mattyb149
Github user mattyb149 commented on the issue:

https://github.com/apache/nifi/pull/2712
  
+1 LGTM, thanks for the updates! Merging to master


---


[GitHub] nifi issue #2712: NIFI-5197 Fixed a bunch of EL scope bugs.

2018-05-17 Thread mattyb149
Github user mattyb149 commented on the issue:

https://github.com/apache/nifi/pull/2712
  
I'm on it...


---


[GitHub] nifi issue #2712: NIFI-5197 Fixed a bunch of EL scope bugs.

2018-05-17 Thread MikeThomsen
Github user MikeThomsen commented on the issue:

https://github.com/apache/nifi/pull/2712
  
@mattyb149 @markap14 @pvillard31 this is semi-blocking 2-3 of my commits. 
They need to be rebased off of it so I can clean up the hack cited in 5145 from 
them. If you could do a quick review, I'd appreciate it.


---


[GitHub] nifi issue #2712: NIFI-5197 Fixed a bunch of EL scope bugs.

2018-05-16 Thread MikeThomsen
Github user MikeThomsen commented on the issue:

https://github.com/apache/nifi/pull/2712
  
@mattyb149 Recreated it. I thought I had fubar'd git, but it turned out to 
just be a sync issue between ASF and GitHub.


---