[jira] [Commented] (NIFI-4904) PutElasticSearch5 should support higher than elasticsearch 5.0.0

2018-07-13 Thread Mike Thomsen (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-4904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543941#comment-16543941
 ] 

Mike Thomsen commented on NIFI-4904:


The new processors defined in 5248 will make this OBE by offloading the work of 
supporting new ES versions to their client service.

> PutElasticSearch5 should support higher than elasticsearch 5.0.0
> 
>
> Key: NIFI-4904
> URL: https://issues.apache.org/jira/browse/NIFI-4904
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Extensions
>Affects Versions: 1.5.0
> Environment: Ubuntu
>Reporter: Dye357
>Priority: Trivial
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> Currently the PutElasticSearch5 component is using the following transport 
> artifact
> 
>  org.elasticsearch.client
>  transport
>  ${es.version}
>  
> Where es.version is 5.0.1. Upgrading to the highest 5.x dependency would 
> enable this component to be compatible with later 5.x versions of elastic 
> search as well as early versions of elastic search 6.x.
> Here is Nifi 1.5.0 connecting to ES 6.2.1 on port 9300:
> [2018-02-23T01:41:04,162][WARN ][o.e.t.n.Netty4Transport ] [uQSW8O8] 
> exception caught on transport layer 
> [NettyTcpChannel\{localAddress=/127.0.0.1:9300, 
> remoteAddress=/127.0.0.1:57457}], closing connection
> java.lang.IllegalStateException: Received message from unsupported version: 
> [5.0.0] minimal compatible version is: [5.6.0]
>  at 
> org.elasticsearch.transport.TcpTransport.ensureVersionCompatibility(TcpTransport.java:1430)
>  ~[elasticsearch-6.2.1.jar:6.2.1]
>  at 
> org.elasticsearch.transport.TcpTransport.messageReceived(TcpTransport.java:1377)
>  ~[elasticsearch-6.2.1.jar:6.2.1]
>  at 
> org.elasticsearch.transport.netty4.Netty4MessageChannelHandler.channelRead(Netty4MessageChannelHandler.java:64)
>  ~[transport-netty4-6.2.1.jar:6.2.1]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:310)
>  [netty-codec-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:297)
>  [netty-codec-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.handler.codec.ByteToMessageDecoder.callDecode(ByteToMessageDecoder.java:413)
>  [netty-codec-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:265)
>  [netty-codec-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.handler.logging.LoggingHandler.channelRead(LoggingHandler.java:241) 
> [netty-handler-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:340)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1359)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:362)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:348)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:935)
>  [netty-transport-4.1.16.Final.jar:4.1.16.Final]
>  at 
> 

[jira] [Commented] (NIFI-3124) Test code cleanup in ElasticSearch (2.x API) processors

2018-07-13 Thread Mike Thomsen (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543940#comment-16543940
 ] 

Mike Thomsen commented on NIFI-3124:


I found a Maven plugin that loads Elastic for us in cases like this. We already 
use it with the elasticsearch restapi bundle. I think it'll be a good fit here 
assuming it supports 2.X.

> Test code cleanup in ElasticSearch (2.x API) processors
> ---
>
> Key: NIFI-3124
> URL: https://issues.apache.org/jira/browse/NIFI-3124
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Extensions
>Reporter: Jeff Storck
>Assignee: Mike Thomsen
>Priority: Minor
>
> Tests that are marked with @Ignore due to their integration nature should be 
> moved into integration tests classes.
> Also, the current integration tests in ITQueryElasticsearchHttp and 
> ITScrollElasticsearchHttp should be updated to not assume hostnames and 
> existing data.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Assigned] (NIFI-3124) Test code cleanup in ElasticSearch (2.x API) processors

2018-07-13 Thread Mike Thomsen (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-3124?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mike Thomsen reassigned NIFI-3124:
--

Assignee: Mike Thomsen

> Test code cleanup in ElasticSearch (2.x API) processors
> ---
>
> Key: NIFI-3124
> URL: https://issues.apache.org/jira/browse/NIFI-3124
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Extensions
>Reporter: Jeff Storck
>Assignee: Mike Thomsen
>Priority: Minor
>
> Tests that are marked with @Ignore due to their integration nature should be 
> moved into integration tests classes.
> Also, the current integration tests in ITQueryElasticsearchHttp and 
> ITScrollElasticsearchHttp should be updated to not assume hostnames and 
> existing data.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5081) Lack of guidance and inability to deal with ISO-8601 dates

2018-07-13 Thread Mike Thomsen (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543934#comment-16543934
 ] 

Mike Thomsen commented on NIFI-5081:


[~forbesmyester] you should be able to use PutMongoRecord for this. What you do 
is you fill out the "timestamp" property of the record reader (ex 
JsonTreeReader) to use the proper date parsing syntax for reading an ISO8601 
string. There's an example somewhere on the Record API document that you can 
lift for some flavors of ISO8601. Then in the Avro schema you declare the field 
to be a long with a logicalType ([Avro 
documentation|https://avro.apache.org/docs/1.8.0/spec.html#Timestamp+%28millisecond+precision%29])

> Lack of guidance and inability to deal with ISO-8601 dates
> --
>
> Key: NIFI-5081
> URL: https://issues.apache.org/jira/browse/NIFI-5081
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Documentation  Website
>Affects Versions: 1.6.0
> Environment: Ubuntu / Chromeium
>Reporter: Matt Forrester
>Priority: Minor
> Attachments: y.xml
>
>
> I've got a Node process that outputs in JSON onto an SQS queue. The dates it 
> spits out are ISO-8601 dates within a string, which is the normal, default 
> and best way to do this in JSON.
> I tried putting them into MongoDB with PutMongo and they go in as strings, 
> which is not good ( https://issues.apache.org/jira/browse/NIFI-2079 ).
> Gave up on Mongo and tried PostgreSQL...
> Figuring I was in Java land I used an esoteric path of GetSQS > 
> EvaluateJsonPath > UpdateAttribute [ 
> "$\{time:toDate("-MM-dd'T'HH:mm:ss.SSS'Z'", "GMT") ] > PutSQL to get it 
> into what I assume is a java.lang.Date, it took me forever to find the 
> sql.args.N.type's required but for some reason PutSQL does not like 
> java.util.Dates.
> Eventually found the ConvertJSONToSQL processor and this created my SQL for 
> me, but it doesn't work as it leaves ISO-8601 dates as ISO-8601 dates, which 
> don't seem to work.
> Eventually found this 
> [https://community.hortonworks.com/questions/84772/putsql-with-date-as-argument.html]
>  and now I have something working, but I'm using my esoteric GetSQS -> 
> EvaluateJsonPath -> UpdateAttribute -> PutSQL path again.
> Think there should be some documentation around this at least because it's 
> very non-obvious.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (NIFI-2135) PutMongo replacing document instead of updating document

2018-07-13 Thread Mike Thomsen (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-2135?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mike Thomsen resolved NIFI-2135.

Resolution: Fixed

[~bende] doing a little cleanup on our Mongo backlog and noticed that 4588 
appears to settle the issue. Feel free to reopen if you disagree.

> PutMongo replacing document instead of updating document
> 
>
> Key: NIFI-2135
> URL: https://issues.apache.org/jira/browse/NIFI-2135
> Project: Apache NiFi
>  Issue Type: Bug
>Affects Versions: 0.6.1
>Reporter: Bryan Bende
>Priority: Minor
> Attachments: 
> 0001-NIFI-2135-PutMongo-will-now-update-using-a-set-and-r.patch, 
> 0002-Verified-insert-update-and-replace.-Fixed-a-minor-lo.patch, 
> nifi-mongodb-nar-0.7.0.nar
>
>
> How can I use put mongo processor to add a new key value pair to an existing 
> document? The flow file contains the document object Id. I have set 'mode' 
> property to 'update' and 'upsert' property to false and 'update query key' 
> property to '_id'. Flow file content is something like this.
> {
> _id:ObjectId(577216f0154b943fe8068079)
> expired:true
> }
> Without inserting the 'expired:true', it replaces the whole document with the 
> given one. So is there  a way to  insert the new key value pair to collection 
> without replacing the whole collection in MongoDB using putmongo processor? 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (NIFI-2079) Cannot insert ISO dates correctly

2018-07-13 Thread Mike Thomsen (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-2079?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mike Thomsen resolved NIFI-2079.

Resolution: Won't Fix

I am closing this for a few reasons:
 # We can insert dates properly using PutMongoRecord.
 # Given the nature of Mongo itself, we cannot blithely assume that a user 
really intended a date string to be automagically parsed into a date object.

If someone has a compelling reason to turn this into a feature request w/ a 
good and intuitive way to let the user determine when to convert and when not 
to, then we can reopen/revisit this.

> Cannot insert ISO dates correctly
> -
>
> Key: NIFI-2079
> URL: https://issues.apache.org/jira/browse/NIFI-2079
> Project: Apache NiFi
>  Issue Type: Bug
>Affects Versions: 0.6.1
>Reporter: asanka sanjaya
>Priority: Major
>
> When try to insert an ISO date to a mongo collection, it always saved as a 
> String instead of ISODate object.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5305) Parameterized collection name fails with dynamic params

2018-07-13 Thread Mike Thomsen (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5305?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543920#comment-16543920
 ] 

Mike Thomsen commented on NIFI-5305:


[~calderonmluis] Your stacktrace shows no value for the collection name. See:

> 'Invalid namespace specified 'my-schema.'' on server mongodb:22277 
> com.mongodb.MongoQueryException: Query failed with error code 73 and error 
> message 'Invalid namespace specified 'my-schema.''

So, as far as I can tell, it should be failing here. If you look at the setup 
method of GetMongoIT in GitHub, you'll see that we actually start with an 
expression language statement like this:

 
{code:java}
runner = TestRunners.newTestRunner(GetMongo.class);
runner.setVariable("uri", MONGO_URI);
runner.setVariable("db", DB_NAME);
runner.setVariable("collection", COLLECTION_NAME);
runner.setProperty(AbstractMongoProcessor.URI, "${uri}");
runner.setProperty(AbstractMongoProcessor.DATABASE_NAME, "${db}");
runner.setProperty(AbstractMongoProcessor.COLLECTION_NAME, "${collection}");
runner.setProperty(GetMongo.USE_PRETTY_PRINTING, GetMongo.YES_PP);
runner.setIncomingConnection(false);
{code}
Can you provide more details about your particular use?

> Parameterized collection name fails with dynamic params
> ---
>
> Key: NIFI-5305
> URL: https://issues.apache.org/jira/browse/NIFI-5305
> Project: Apache NiFi
>  Issue Type: Bug
>Affects Versions: 1.6.0
>Reporter: Luis M Calderon
>Assignee: Mike Thomsen
>Priority: Major
> Attachments: getmongo_error.txt
>
>
> When using a dynamic property for collection name, the creation of the full 
> query reference name is not working as expected. The reference name seems to 
> be the concatenation of the namespace and collection name. i.e. 
> _.)_ 
> The following parameters are those which are provided to the GetMongo 
> Processor:
> Param Definitions:
>     my_schema = 'my_schema'
>     collection_name = 
> {code:java}
> Mongo Database Name: ${my_schema}
> Mongo Collection Name: ${collection_name}{code}
>  
> Attached is a text file containing the NiFi log for this error.
>  
> _There is no namespace error when the same values are hard coded._
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5196) AbstractJMSProcessor can leave connection hanging open

2018-07-13 Thread Nick Coleman (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5196?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543862#comment-16543862
 ] 

Nick Coleman commented on NIFI-5196:


[~mosermw] - this would satisfy the original issue of needing to reboot the 
server knowing that the admin has to stop and restart the processor.  It's also 
a cleaner code change

> AbstractJMSProcessor can leave connection hanging open
> --
>
> Key: NIFI-5196
> URL: https://issues.apache.org/jira/browse/NIFI-5196
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Extensions
>Affects Versions: 1.6.0
>Reporter: Nick Coleman
>Assignee: Sivaprasanna Sethuraman
>Priority: Major
>  Labels: JMS
>
> ConsumeJMS and PublishJMS are based on AbstractJMSProcessor.  They can cause 
> a connection to the MQ Server to be opened and not be closed until the NiFi 
> server is rebooted.
> This can create a problem for an MQ when the initial setup entered is wrong 
> for an IBM MQ system that only allows one connection per user.  Subsequent 
> connections are blocked as the first remains open.  Another potential problem 
> even if the subsequent connection works is the original connection is still 
> open and taking up resources.
> A simple change to the AbstractJMSProcessor would be in the onTrigger() 
> function:
>  
> {code:java}
> @Override
> public void onTrigger(ProcessContext context, ProcessSession session) throws 
> ProcessException {
> T worker = workerPool.poll();
> if (worker == null) {
> worker = buildTargetResource(context);
> }
> boolean offered = false;
> try {
> rendezvousWithJms(context, session, worker);
> offered = workerPool.offer(worker);
> }
> finally {
> if (!offered) {
> worker.shutdown();
> }
> }
> }{code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-3672) Older versions of IBM MQ allowed integer value to be set as String

2018-07-13 Thread Michael Moser (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-3672?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543624#comment-16543624
 ] 

Michael Moser commented on NIFI-3672:
-

+1 for adding this feature to PublishJMS.

The deprecated PutJMS processor supported the {{javax.jms.Message}} methods 
{{setIntProperty}}, {{setBooleanProperty}}, {{setShortProperty}}, 
{{setLongProperty}}, {{setByteProperty}}, {{setDouleProperty}} and 
{{setFloatProperty}}.  PublishJMS could also support them.

> Older versions of IBM MQ allowed integer value to be set as String
> --
>
> Key: NIFI-3672
> URL: https://issues.apache.org/jira/browse/NIFI-3672
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Extensions
>Affects Versions: 1.1.1
>Reporter: Andy LoPresto
>Priority: Minor
>  Labels: beginner, jms, third-party
>
> As reported in this [Stack Overflow 
> question|https://stackoverflow.com/questions/43213416/publishjms-processor-failing-for-writing-message-to-ibm-websphere-mq],
>  the {{PublishJMS}} processor was failing to connect to an IBM MQ JMS server 
> with the exception {{com.ibm.msg.client.jms.DetailedMessageFormatException: 
> JMSCC0051: The property 'JMS_IBM_MsgType' should be set using type 
> 'java.lang.Integer', not 'java.lang.String'.}} As noted in the answer, this 
> is a [known IBM issue 
> IT02814|https://www-01.ibm.com/support/docview.wss?uid=swg1IT02814] in which 
> older (pre-7.0) versions allowed integer properties to be set as a 
> {{java.lang.String}} but new versions require it to be a 
> {{java.lang.Integer}}. The property descriptor should be changed to validate 
> an integer value and return it with the correct type when setting the 
> configuration. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-4535) Set the Page Title to the name of the Root Process Group

2018-07-13 Thread Peter Wicks (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-4535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543600#comment-16543600
 ] 

Peter Wicks commented on NIFI-4535:
---

Thanks, I'll fix it.

> Set the Page Title to the name of the Root Process Group
> 
>
> Key: NIFI-4535
> URL: https://issues.apache.org/jira/browse/NIFI-4535
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Core UI
>Reporter: Peter Wicks
>Assignee: Peter Wicks
>Priority: Minor
> Fix For: 1.8.0
>
>
> NiFi's UI has a hard coded page title of NiFi. I have many servers and it's 
> hard to keep track of the tabs in Chrome.
> Please change the Title of the page so it matches the name of the Root 
> processor group. This way I can name the root group by server/instance, and 
> easily identify the right tab/window.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (NIFI-5429) Improve failure monitoring

2018-07-13 Thread Andy LoPresto (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5429?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andy LoPresto updated NIFI-5429:

Attachment: asf_hc_failure_reporting.txt

> Improve failure monitoring
> --
>
> Key: NIFI-5429
> URL: https://issues.apache.org/jira/browse/NIFI-5429
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Core Framework
>Affects Versions: 1.7.0
>Reporter: Andy LoPresto
>Priority: Major
>  Labels: failure, monitoring, provenance, reporting
> Attachments: asf_hc_failure_reporting.001.jpeg, 
> asf_hc_failure_reporting.002.jpeg, asf_hc_failure_reporting.003.jpeg, 
> asf_hc_failure_reporting.txt
>
>
> A user in the ASF HipChat room and I discussed the following scenario:
> {quote}
> With N processors in a flow, I want to send a notification on any failure 
> event to a remote monitoring system. For this example, let the remote 
> monitoring system be Elasticsearch. Currently, I need N unique 
> PutElasticsearch processors to uniquely identify the processor which has the 
> failure. 
> {quote}
> I created some diagrams showing the existing setup and desired setup (one 
> with direct to reporting; one with an {{ExecuteScript}} processor for 
> metadata enrichment (i.e. if the component {{UUID}} is available, use {{ES}} 
> to resolve the human-friendly {{name}} and processor group nesting to be 
> persisted)). 
> There are concerns about exposing component ID to flowfiles and how this can 
> be misused, but there needs to be a better solution for how to monitor flow 
> failures without 1:1 processors for monitoring. 
> I have also attached the conversation. 
> cc: [~markap14] [~joewitt]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (NIFI-5429) Improve failure monitoring

2018-07-13 Thread Andy LoPresto (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5429?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andy LoPresto updated NIFI-5429:

Attachment: asf_hc_failure_reporting.003.jpeg
asf_hc_failure_reporting.002.jpeg
asf_hc_failure_reporting.001.jpeg

> Improve failure monitoring
> --
>
> Key: NIFI-5429
> URL: https://issues.apache.org/jira/browse/NIFI-5429
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Core Framework
>Affects Versions: 1.7.0
>Reporter: Andy LoPresto
>Priority: Major
>  Labels: failure, monitoring, provenance, reporting
> Attachments: asf_hc_failure_reporting.001.jpeg, 
> asf_hc_failure_reporting.002.jpeg, asf_hc_failure_reporting.003.jpeg
>
>
> A user in the ASF HipChat room and I discussed the following scenario:
> {quote}
> With N processors in a flow, I want to send a notification on any failure 
> event to a remote monitoring system. For this example, let the remote 
> monitoring system be Elasticsearch. Currently, I need N unique 
> PutElasticsearch processors to uniquely identify the processor which has the 
> failure. 
> {quote}
> I created some diagrams showing the existing setup and desired setup (one 
> with direct to reporting; one with an {{ExecuteScript}} processor for 
> metadata enrichment (i.e. if the component {{UUID}} is available, use {{ES}} 
> to resolve the human-friendly {{name}} and processor group nesting to be 
> persisted)). 
> There are concerns about exposing component ID to flowfiles and how this can 
> be misused, but there needs to be a better solution for how to monitor flow 
> failures without 1:1 processors for monitoring. 
> I have also attached the conversation. 
> cc: [~markap14] [~joewitt]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (NIFI-5429) Improve failure monitoring

2018-07-13 Thread Andy LoPresto (JIRA)
Andy LoPresto created NIFI-5429:
---

 Summary: Improve failure monitoring
 Key: NIFI-5429
 URL: https://issues.apache.org/jira/browse/NIFI-5429
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Core Framework
Affects Versions: 1.7.0
Reporter: Andy LoPresto


A user in the ASF HipChat room and I discussed the following scenario:

{quote}
With N processors in a flow, I want to send a notification on any failure event 
to a remote monitoring system. For this example, let the remote monitoring 
system be Elasticsearch. Currently, I need N unique PutElasticsearch processors 
to uniquely identify the processor which has the failure. 
{quote}

I created some diagrams showing the existing setup and desired setup (one with 
direct to reporting; one with an {{ExecuteScript}} processor for metadata 
enrichment (i.e. if the component {{UUID}} is available, use {{ES}} to resolve 
the human-friendly {{name}} and processor group nesting to be persisted)). 

There are concerns about exposing component ID to flowfiles and how this can be 
misused, but there needs to be a better solution for how to monitor flow 
failures without 1:1 processors for monitoring. 

I have also attached the conversation. 

cc: [~markap14] [~joewitt]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-4535) Set the Page Title to the name of the Root Process Group

2018-07-13 Thread Matt Gilman (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-4535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543589#comment-16543589
 ] 

Matt Gilman commented on NIFI-4535:
---

In the same place where the document title was being set, we have similar code 
to set the current group name.
{code:java}
if (breadcrumb.permissions.canRead) {
nfCanvas.setGroupName(breadcrumb.breadcrumb.name);
} else {
nfCanvas.setGroupName(breadcrumb.id);
}
{code}

> Set the Page Title to the name of the Root Process Group
> 
>
> Key: NIFI-4535
> URL: https://issues.apache.org/jira/browse/NIFI-4535
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Core UI
>Reporter: Peter Wicks
>Assignee: Peter Wicks
>Priority: Minor
> Fix For: 1.8.0
>
>
> NiFi's UI has a hard coded page title of NiFi. I have many servers and it's 
> hard to keep track of the tabs in Chrome.
> Please change the Title of the page so it matches the name of the Root 
> processor group. This way I can name the root group by server/instance, and 
> easily identify the right tab/window.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-4889) Logout not working properly with OIDC

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-4889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543572#comment-16543572
 ] 

ASF GitHub Bot commented on NIFI-4889:
--

Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/2830
  
@Trojan295 Thanks for the PR! The code looks good but I ran into an issue 
when attempting to log out. Despite being a required field in the openid spec 
[1], it appears that at least in practice [2] the `end_session_endpoint` field 
is not guaranteed to be present. I think we may need to account for this 
possibility in the logout endpoint.

[1] https://openid.net/specs/openid-connect-session-1_0.html#OPMetadata
[2] https://accounts.google.com/.well-known/openid-configuration


> Logout not working properly with OIDC
> -
>
> Key: NIFI-4889
> URL: https://issues.apache.org/jira/browse/NIFI-4889
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Core UI
>Affects Versions: 1.5.0
> Environment: Browser: Chrome / Firefox
> Configuration of NiFi:
> - SSL certificate for the server (no client auth)
> - OIDC configuration including end_session_endpoint (see the link 
> https://auth.s.orchestracities.com/auth/realms/default/.well-known/openid-configuration)
>Reporter: Federico Michele Facca
>Priority: Critical
>
> Click on logout, i would expect to logout and getting redirect to the auth 
> page. But given that the session is not closed on the oauth provider, i get 
> logged in again.
> I suppose the solution would be to invoke the end_session_endpoint provided 
> in the openid discovery configuration.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi issue #2830: NIFI-4889: Logout not working properly with OIDC

2018-07-13 Thread mcgilman
Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/2830
  
@Trojan295 Thanks for the PR! The code looks good but I ran into an issue 
when attempting to log out. Despite being a required field in the openid spec 
[1], it appears that at least in practice [2] the `end_session_endpoint` field 
is not guaranteed to be present. I think we may need to account for this 
possibility in the logout endpoint.

[1] https://openid.net/specs/openid-connect-session-1_0.html#OPMetadata
[2] https://accounts.google.com/.well-known/openid-configuration


---


[jira] [Commented] (NIFI-4535) Set the Page Title to the name of the Root Process Group

2018-07-13 Thread Matt Gilman (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-4535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543568#comment-16543568
 ] 

Matt Gilman commented on NIFI-4535:
---

[~patricker] I just ran into an issue that I think is related to this PR. If 
the user doesn't have permissions to a group, the BreadcrumbDTO on the 
BreadcrumbEntity is not populated. As a result, we hit a JS error that prevents 
the page from loading. I'm thinking the page name detection needs to be updated 
to account for this.

> Set the Page Title to the name of the Root Process Group
> 
>
> Key: NIFI-4535
> URL: https://issues.apache.org/jira/browse/NIFI-4535
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Core UI
>Reporter: Peter Wicks
>Assignee: Peter Wicks
>Priority: Minor
> Fix For: 1.8.0
>
>
> NiFi's UI has a hard coded page title of NiFi. I have many servers and it's 
> hard to keep track of the tabs in Chrome.
> Please change the Title of the page so it matches the name of the Root 
> processor group. This way I can name the root group by server/instance, and 
> easily identify the right tab/window.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Reopened] (NIFI-4535) Set the Page Title to the name of the Root Process Group

2018-07-13 Thread Matt Gilman (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-4535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Gilman reopened NIFI-4535:
---

> Set the Page Title to the name of the Root Process Group
> 
>
> Key: NIFI-4535
> URL: https://issues.apache.org/jira/browse/NIFI-4535
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Core UI
>Reporter: Peter Wicks
>Assignee: Peter Wicks
>Priority: Minor
> Fix For: 1.8.0
>
>
> NiFi's UI has a hard coded page title of NiFi. I have many servers and it's 
> hard to keep track of the tabs in Chrome.
> Please change the Title of the page so it matches the name of the Root 
> processor group. This way I can name the root group by server/instance, and 
> easily identify the right tab/window.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5426) Use NIO.2 API for ListFile to avoid multiple disk reads

2018-07-13 Thread Mark Payne (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543520#comment-16543520
 ] 

Mark Payne commented on NIFI-5426:
--

[~mgaido] thanks for knocking that out! I was able to test and verify that 
things appear to work well on my system. Code review all looks good as well. +1 
merged to master. Thanks again!

> Use NIO.2 API for ListFile to avoid multiple disk reads
> ---
>
> Key: NIFI-5426
> URL: https://issues.apache.org/jira/browse/NIFI-5426
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Extensions
>Reporter: Mark Payne
>Priority: Major
> Fix For: 1.8.0
>
>
> NIFI-4631 attempted to do something similar but had issues when running 
> against CIFS mounts, reportedly. However, I think the issue was not the use 
> necessarily of NIO.2 but the way in which it was used. It attempted to use 
> walkTree and did did not appear to follow symlinks. As a result, this caused 
> failures in some scenarios. We should look at using NIO.2 and probably obtain 
> a Stream.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (NIFI-5426) Use NIO.2 API for ListFile to avoid multiple disk reads

2018-07-13 Thread Mark Payne (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5426?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mark Payne updated NIFI-5426:
-
Fix Version/s: 1.8.0
   Status: Patch Available  (was: Open)

> Use NIO.2 API for ListFile to avoid multiple disk reads
> ---
>
> Key: NIFI-5426
> URL: https://issues.apache.org/jira/browse/NIFI-5426
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Extensions
>Reporter: Mark Payne
>Priority: Major
> Fix For: 1.8.0
>
>
> NIFI-4631 attempted to do something similar but had issues when running 
> against CIFS mounts, reportedly. However, I think the issue was not the use 
> necessarily of NIO.2 but the way in which it was used. It attempted to use 
> walkTree and did did not appear to follow symlinks. As a result, this caused 
> failures in some scenarios. We should look at using NIO.2 and probably obtain 
> a Stream.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5426) Use NIO.2 API for ListFile to avoid multiple disk reads

2018-07-13 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543519#comment-16543519
 ] 

ASF subversion and git services commented on NIFI-5426:
---

Commit 31bb89514cfa0e5df9db578387d566c7b5117c45 in nifi's branch 
refs/heads/master from [~mgaido]
[ https://git-wip-us.apache.org/repos/asf?p=nifi.git;h=31bb895 ]

NIFI-5426: Use NIO.2 API for ListFile


> Use NIO.2 API for ListFile to avoid multiple disk reads
> ---
>
> Key: NIFI-5426
> URL: https://issues.apache.org/jira/browse/NIFI-5426
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Extensions
>Reporter: Mark Payne
>Priority: Major
>
> NIFI-4631 attempted to do something similar but had issues when running 
> against CIFS mounts, reportedly. However, I think the issue was not the use 
> necessarily of NIO.2 but the way in which it was used. It attempted to use 
> walkTree and did did not appear to follow symlinks. As a result, this caused 
> failures in some scenarios. We should look at using NIO.2 and probably obtain 
> a Stream.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5196) AbstractJMSProcessor can leave connection hanging open

2018-07-13 Thread Michael Moser (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5196?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543485#comment-16543485
 ] 

Michael Moser commented on NIFI-5196:
-

I have another scenario when this happens.  If the framework throws an 
exception from the ProcessSession commit() method, then that exception 
propagates out of  rendezvousWithJms() and the AbstractJMSProcessor 
onTrigger().  Situations like an OutOfMemoryError or IOException: No space left 
on device can cause this.

In this situation, I don't think we should close and reopen connections to the 
JMS hub in each onTrigger.  So I propose a solution like this:
{code:java}
@Override
public void onTrigger(ProcessContext context, ProcessSession session) throws 
ProcessException {
T worker = workerPool.poll();
if (worker == null) {
worker = buildTargetResource(context);
}

try {
rendezvousWithJms(context, session, worker);
}
finally {
workerPool.offer(worker);
}
}
{code}
In the case where OOM or out of disk space occurs, the valid connection simply 
remains open. In the case where the connection configuration is wrong, the NiFi 
manager should be able to notice the problem and stop the processor, which will 
call the @OnStopped method and close the bad connection.

How does this sound?

> AbstractJMSProcessor can leave connection hanging open
> --
>
> Key: NIFI-5196
> URL: https://issues.apache.org/jira/browse/NIFI-5196
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Extensions
>Affects Versions: 1.6.0
>Reporter: Nick Coleman
>Assignee: Sivaprasanna Sethuraman
>Priority: Major
>  Labels: JMS
>
> ConsumeJMS and PublishJMS are based on AbstractJMSProcessor.  They can cause 
> a connection to the MQ Server to be opened and not be closed until the NiFi 
> server is rebooted.
> This can create a problem for an MQ when the initial setup entered is wrong 
> for an IBM MQ system that only allows one connection per user.  Subsequent 
> connections are blocked as the first remains open.  Another potential problem 
> even if the subsequent connection works is the original connection is still 
> open and taking up resources.
> A simple change to the AbstractJMSProcessor would be in the onTrigger() 
> function:
>  
> {code:java}
> @Override
> public void onTrigger(ProcessContext context, ProcessSession session) throws 
> ProcessException {
> T worker = workerPool.poll();
> if (worker == null) {
> worker = buildTargetResource(context);
> }
> boolean offered = false;
> try {
> rendezvousWithJms(context, session, worker);
> offered = workerPool.offer(worker);
> }
> finally {
> if (!offered) {
> worker.shutdown();
> }
> }
> }{code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi issue #2884: NIFI-3993 Updated the ZooKeeper version to 3.4.10

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

https://github.com/apache/nifi/pull/2884
  
@joewitt I brought up the upgrade a while ago, but many of the 
services/components we integrate with are using Zookeeper 3.4.6.  Originally I 
wanted to use a newer version that did not explicitly depend on log4j, because 
it was causing some issues with the NiFi Toolkit.  Since we were able to work 
around the dependency issue, we didn't upgrade the version.


---


[jira] [Commented] (NIFI-3993) Upgrade embedded ZooKeeper version

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-3993?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543452#comment-16543452
 ] 

ASF GitHub Bot commented on NIFI-3993:
--

Github user jtstorck commented on the issue:

https://github.com/apache/nifi/pull/2884
  
@joewitt I brought up the upgrade a while ago, but many of the 
services/components we integrate with are using Zookeeper 3.4.6.  Originally I 
wanted to use a newer version that did not explicitly depend on log4j, because 
it was causing some issues with the NiFi Toolkit.  Since we were able to work 
around the dependency issue, we didn't upgrade the version.


> Upgrade embedded ZooKeeper version
> --
>
> Key: NIFI-3993
> URL: https://issues.apache.org/jira/browse/NIFI-3993
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Core Framework
>Affects Versions: 1.2.0
>Reporter: Mark Bean
>Priority: Major
>
> In a Cluster configuration, Nodes are periodically disconnected from the 
> Cluster, and then reconnected. These events correspond to the following error:
> ERROR [CommitProcessor:1] o.apache.zookeeper.server.NIOServerCnxn Unexpected 
> Exception:
> java.nio.channels.CancelledKeyException: null
> at sun.nio.ch.SelectionKeyImpl.ensureValid(SectionKeyImpl.java:73)
> at sun.nio.ch.SelectionKeyImpl.interestOps(SelctionKeyImpl.java:77)
> at 
> org.apache.zookeeper.server.NIOServerCnxn.sendBuffer(NIOServerCnxn.java:151)
> at 
> org.apache.zookeeper.server.NIOServerCnXn.sendResopnse(NIOServerCnxn.java:1081)
> at 
> org.apache.zookeeper.server.FinalRequestProcessor.processRequest(FinalRequestProcessor.java:404)
> at 
> org.apache.zookeeper.server.quorum.CommitProcessor.run(CommitProcessor.java:74)
> This error was reported in ZooKeeper JIRA [1], and reported as fixed in 
> version 3.4.10, the current stable build. As additional confirmation, when 
> using a stand-alone ZK, 3.4.10, rather than the embedded ZK, the above error 
> was no longer observed.
> Update NiFi to use ZK 3.4.10
> [1] https://issues.apache.org/jira/browse/ZOOKEEPER-2044



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (NIFI-5428) Add documentation for S2S and Flow Analyzer tools

2018-07-13 Thread Andrew Lim (JIRA)
Andrew Lim created NIFI-5428:


 Summary: Add documentation for S2S and Flow Analyzer tools
 Key: NIFI-5428
 URL: https://issues.apache.org/jira/browse/NIFI-5428
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Documentation  Website
Reporter: Andrew Lim


I noticed that these two Admin toolkit tools are not documented.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Assigned] (NIFI-5305) Parameterized collection name fails with dynamic params

2018-07-13 Thread Mike Thomsen (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5305?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mike Thomsen reassigned NIFI-5305:
--

Assignee: Mike Thomsen

> Parameterized collection name fails with dynamic params
> ---
>
> Key: NIFI-5305
> URL: https://issues.apache.org/jira/browse/NIFI-5305
> Project: Apache NiFi
>  Issue Type: Bug
>Affects Versions: 1.6.0
>Reporter: Luis M Calderon
>Assignee: Mike Thomsen
>Priority: Major
> Attachments: getmongo_error.txt
>
>
> When using a dynamic property for collection name, the creation of the full 
> query reference name is not working as expected. The reference name seems to 
> be the concatenation of the namespace and collection name. i.e. 
> _.)_ 
> The following parameters are those which are provided to the GetMongo 
> Processor:
> Param Definitions:
>     my_schema = 'my_schema'
>     collection_name = 
> {code:java}
> Mongo Database Name: ${my_schema}
> Mongo Collection Name: ${collection_name}{code}
>  
> Attached is a text file containing the NiFi log for this error.
>  
> _There is no namespace error when the same values are hard coded._
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5427) ScrollElasticsearchHttp breaks with Elasticsearch 6

2018-07-13 Thread Joseph Gresock (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543416#comment-16543416
 ] 

Joseph Gresock commented on NIFI-5427:
--

I have tested this in a live 1.6.0 nifi instance.

> ScrollElasticsearchHttp breaks with Elasticsearch 6
> ---
>
> Key: NIFI-5427
> URL: https://issues.apache.org/jira/browse/NIFI-5427
> Project: Apache NiFi
>  Issue Type: Bug
>Affects Versions: 1.4.0, 1.5.0, 1.6.0, 1.7.0
>Reporter: Joseph Gresock
>Assignee: Joseph Gresock
>Priority: Minor
>
> The Elasticsearch 6 Scroll API uses scroll IDs that are nearly 10,000 
> characters long.  Since the ScrollElasticsearchHttp processor uses a GET 
> request for scrolling, this breaks the processor after the first scroll call. 
>  
> The solution is fairly simple: use a POST instead of a GET for the scroll 
> calls.  This is backwards compatible down to Elasticsearch 2.X.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5427) ScrollElasticsearchHttp breaks with Elasticsearch 6

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543414#comment-16543414
 ] 

ASF GitHub Bot commented on NIFI-5427:
--

GitHub user gresockj opened a pull request:

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

NIFI-5427: Updating ScrollElasticsearchHttp to use POST, supporting ES6

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?
- [x] Have you written or updated unit tests to verify your changes?
- [ ] 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/gresockj/nifi NIFI-5427

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

https://github.com/apache/nifi/pull/2890.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 #2890


commit 2b15a7ac2e91b8bf6d922cb3c7e9eb682bc7b3ff
Author: Joe Gresock 
Date:   2018-07-13T16:29:45Z

NIFI-5427: Updating ScrollElasticsearchHttp to use POST, supporting ES6




> ScrollElasticsearchHttp breaks with Elasticsearch 6
> ---
>
> Key: NIFI-5427
> URL: https://issues.apache.org/jira/browse/NIFI-5427
> Project: Apache NiFi
>  Issue Type: Bug
>Affects Versions: 1.4.0, 1.5.0, 1.6.0, 1.7.0
>Reporter: Joseph Gresock
>Assignee: Joseph Gresock
>Priority: Minor
>
> The Elasticsearch 6 Scroll API uses scroll IDs that are nearly 10,000 
> characters long.  Since the ScrollElasticsearchHttp processor uses a GET 
> request for scrolling, this breaks the processor after the first scroll call. 
>  
> The solution is fairly simple: use a POST instead of a GET for the scroll 
> calls.  This is backwards compatible down to Elasticsearch 2.X.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi pull request #2890: NIFI-5427: Updating ScrollElasticsearchHttp to use ...

2018-07-13 Thread gresockj
GitHub user gresockj opened a pull request:

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

NIFI-5427: Updating ScrollElasticsearchHttp to use POST, supporting ES6

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?
- [x] Have you written or updated unit tests to verify your changes?
- [ ] 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/gresockj/nifi NIFI-5427

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

https://github.com/apache/nifi/pull/2890.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 #2890


commit 2b15a7ac2e91b8bf6d922cb3c7e9eb682bc7b3ff
Author: Joe Gresock 
Date:   2018-07-13T16:29:45Z

NIFI-5427: Updating ScrollElasticsearchHttp to use POST, supporting ES6




---


[jira] [Commented] (NIFI-4889) Logout not working properly with OIDC

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-4889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543388#comment-16543388
 ] 

ASF GitHub Bot commented on NIFI-4889:
--

Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/2830
  
Will review...


> Logout not working properly with OIDC
> -
>
> Key: NIFI-4889
> URL: https://issues.apache.org/jira/browse/NIFI-4889
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Core UI
>Affects Versions: 1.5.0
> Environment: Browser: Chrome / Firefox
> Configuration of NiFi:
> - SSL certificate for the server (no client auth)
> - OIDC configuration including end_session_endpoint (see the link 
> https://auth.s.orchestracities.com/auth/realms/default/.well-known/openid-configuration)
>Reporter: Federico Michele Facca
>Priority: Critical
>
> Click on logout, i would expect to logout and getting redirect to the auth 
> page. But given that the session is not closed on the oauth provider, i get 
> logged in again.
> I suppose the solution would be to invoke the end_session_endpoint provided 
> in the openid discovery configuration.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi issue #2830: NIFI-4889: Logout not working properly with OIDC

2018-07-13 Thread mcgilman
Github user mcgilman commented on the issue:

https://github.com/apache/nifi/pull/2830
  
Will review...


---


[jira] [Created] (NIFI-5427) ScrollElasticsearchHttp breaks with Elasticsearch 6

2018-07-13 Thread Joseph Gresock (JIRA)
Joseph Gresock created NIFI-5427:


 Summary: ScrollElasticsearchHttp breaks with Elasticsearch 6
 Key: NIFI-5427
 URL: https://issues.apache.org/jira/browse/NIFI-5427
 Project: Apache NiFi
  Issue Type: Bug
Affects Versions: 1.7.0, 1.6.0, 1.5.0, 1.4.0
Reporter: Joseph Gresock
Assignee: Joseph Gresock


The Elasticsearch 6 Scroll API uses scroll IDs that are nearly 10,000 
characters long.  Since the ScrollElasticsearchHttp processor uses a GET 
request for scrolling, this breaks the processor after the first scroll call. 

 

The solution is fairly simple: use a POST instead of a GET for the scroll 
calls.  This is backwards compatible down to Elasticsearch 2.X.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (MINIFICPP-556) Create snap build

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/MINIFICPP-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543329#comment-16543329
 ] 

ASF GitHub Bot commented on MINIFICPP-556:
--

GitHub user achristianson opened a pull request:

https://github.com/apache/nifi-minifi-cpp/pull/371

MINIFICPP-556 Added initial snapcraft build

This wouldn't have been possible without the help of Jeremy Dyer.

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 MINIFI- 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] 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)?
- [x] If applicable, have you updated the LICENSE file?
- [x] If applicable, have you updated the NOTICE file?

### For documentation related changes:
- [x] 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/achristianson/nifi-minifi-cpp MINIFICPP-556

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

https://github.com/apache/nifi-minifi-cpp/pull/371.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 #371


commit 4c5481aa0c44cf08786326407598659b9f3576c7
Author: Andrew I. Christianson 
Date:   2018-07-13T15:17:37Z

MINIFICPP-556 Added initial snapcraft build




> Create snap build
> -
>
> Key: MINIFICPP-556
> URL: https://issues.apache.org/jira/browse/MINIFICPP-556
> Project: NiFi MiNiFi C++
>  Issue Type: Improvement
>Reporter: Andrew Christianson
>Assignee: Andrew Christianson
>Priority: Major
>
> Package nifi-minifi-cpp into a snap package for easy distribution to users 
> who use snap.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi-minifi-cpp pull request #371: MINIFICPP-556 Added initial snapcraft bui...

2018-07-13 Thread achristianson
GitHub user achristianson opened a pull request:

https://github.com/apache/nifi-minifi-cpp/pull/371

MINIFICPP-556 Added initial snapcraft build

This wouldn't have been possible without the help of Jeremy Dyer.

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 MINIFI- 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] 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)?
- [x] If applicable, have you updated the LICENSE file?
- [x] If applicable, have you updated the NOTICE file?

### For documentation related changes:
- [x] 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/achristianson/nifi-minifi-cpp MINIFICPP-556

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

https://github.com/apache/nifi-minifi-cpp/pull/371.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 #371


commit 4c5481aa0c44cf08786326407598659b9f3576c7
Author: Andrew I. Christianson 
Date:   2018-07-13T15:17:37Z

MINIFICPP-556 Added initial snapcraft build




---


[jira] [Updated] (MINIFICPP-557) Segfault when including C2 AgentInformation in Alpine

2018-07-13 Thread Aldrin Piri (JIRA)


 [ 
https://issues.apache.org/jira/browse/MINIFICPP-557?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aldrin Piri updated MINIFICPP-557:
--
Attachment: minifi-app.log

> Segfault when including C2 AgentInformation in Alpine
> -
>
> Key: MINIFICPP-557
> URL: https://issues.apache.org/jira/browse/MINIFICPP-557
> Project: NiFi MiNiFi C++
>  Issue Type: Bug
>Reporter: Aldrin Piri
>Priority: Major
> Attachments: minifi-app.log
>
>
> When running an instance with AgentInformation of one of the 
> nifi.c2.root.classes, the instance fails to startup with a segfault.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (MINIFICPP-557) Segfault when including C2 AgentInformation in Alpine

2018-07-13 Thread Aldrin Piri (JIRA)


[ 
https://issues.apache.org/jira/browse/MINIFICPP-557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543268#comment-16543268
 ] 

Aldrin Piri commented on MINIFICPP-557:
---

Debug log attached.  This is for an empty configuration.

> Segfault when including C2 AgentInformation in Alpine
> -
>
> Key: MINIFICPP-557
> URL: https://issues.apache.org/jira/browse/MINIFICPP-557
> Project: NiFi MiNiFi C++
>  Issue Type: Bug
>Reporter: Aldrin Piri
>Priority: Major
> Attachments: minifi-app.log
>
>
> When running an instance with AgentInformation of one of the 
> nifi.c2.root.classes, the instance fails to startup with a segfault.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (MINIFICPP-557) Segfault when including C2 AgentInformation in Alpine

2018-07-13 Thread Aldrin Piri (JIRA)
Aldrin Piri created MINIFICPP-557:
-

 Summary: Segfault when including C2 AgentInformation in Alpine
 Key: MINIFICPP-557
 URL: https://issues.apache.org/jira/browse/MINIFICPP-557
 Project: NiFi MiNiFi C++
  Issue Type: Bug
Reporter: Aldrin Piri


When running an instance with AgentInformation of one of the 
nifi.c2.root.classes, the instance fails to startup with a segfault.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5426) Use NIO.2 API for ListFile to avoid multiple disk reads

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543221#comment-16543221
 ] 

ASF GitHub Bot commented on NIFI-5426:
--

GitHub user mgaido91 opened a pull request:

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

NIFI-5426: Use NIO.2 API for ListFile

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:
- [ ] Is there a JIRA ticket associated with this PR? Is it referenced 
 in the commit message?

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

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

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

### For code changes:
- [ ] 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?
- [ ] 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/mgaido91/nifi NIFI-5426

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

https://github.com/apache/nifi/pull/2889.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 #2889


commit 1ce4b51d91d7101d40f59f462929d637adb2c84d
Author: Marco Gaido 
Date:   2018-07-13T14:00:02Z

NIFI-5426: Use NIO.2 API for ListFile




> Use NIO.2 API for ListFile to avoid multiple disk reads
> ---
>
> Key: NIFI-5426
> URL: https://issues.apache.org/jira/browse/NIFI-5426
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Extensions
>Reporter: Mark Payne
>Priority: Major
>
> NIFI-4631 attempted to do something similar but had issues when running 
> against CIFS mounts, reportedly. However, I think the issue was not the use 
> necessarily of NIO.2 but the way in which it was used. It attempted to use 
> walkTree and did did not appear to follow symlinks. As a result, this caused 
> failures in some scenarios. We should look at using NIO.2 and probably obtain 
> a Stream.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi pull request #2889: NIFI-5426: Use NIO.2 API for ListFile

2018-07-13 Thread mgaido91
GitHub user mgaido91 opened a pull request:

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

NIFI-5426: Use NIO.2 API for ListFile

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:
- [ ] Is there a JIRA ticket associated with this PR? Is it referenced 
 in the commit message?

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

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

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

### For code changes:
- [ ] 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?
- [ ] 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/mgaido91/nifi NIFI-5426

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

https://github.com/apache/nifi/pull/2889.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 #2889


commit 1ce4b51d91d7101d40f59f462929d637adb2c84d
Author: Marco Gaido 
Date:   2018-07-13T14:00:02Z

NIFI-5426: Use NIO.2 API for ListFile




---


[jira] [Updated] (NIFI-5415) Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy

2018-07-13 Thread Matt Burgess (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5415?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Burgess updated NIFI-5415:
---
Resolution: Fixed
Status: Resolved  (was: Patch Available)

> Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy
> --
>
> Key: NIFI-5415
> URL: https://issues.apache.org/jira/browse/NIFI-5415
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Tools and Build
>Affects Versions: 1.7.0
>Reporter: Andy LoPresto
>Assignee: Andy LoPresto
>Priority: Blocker
>  Labels: groovy, integration-test, maven, syslog, test
> Fix For: 1.8.0, 1.7.1
>
>
> [~joewitt] made an effort to move inconsistent and network/timing-dependent 
> unit tests to integration tests via NIFI-4659, NIFI-4660, NIFI-4662, and 
> NIFI-4664. 
> This file was renamed (from {{ListenSyslogGroovyTest.groovy}} to 
> {{ITListenSyslogGroovy.groovy}}) but the class name inside was not changed. I 
> believe the maven filter is on the class name, not the file name, because 
> this test still runs in normal builds. It breaks inconsistently on my 
> machine, so I renamed the class to ensure it is not run as part of the usual 
> unit test battery. This is required for 1.7.1 to be released. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5415) Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543220#comment-16543220
 ] 

ASF GitHub Bot commented on NIFI-5415:
--

Github user asfgit closed the pull request at:

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


> Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy
> --
>
> Key: NIFI-5415
> URL: https://issues.apache.org/jira/browse/NIFI-5415
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Tools and Build
>Affects Versions: 1.7.0
>Reporter: Andy LoPresto
>Assignee: Andy LoPresto
>Priority: Blocker
>  Labels: groovy, integration-test, maven, syslog, test
> Fix For: 1.8.0, 1.7.1
>
>
> [~joewitt] made an effort to move inconsistent and network/timing-dependent 
> unit tests to integration tests via NIFI-4659, NIFI-4660, NIFI-4662, and 
> NIFI-4664. 
> This file was renamed (from {{ListenSyslogGroovyTest.groovy}} to 
> {{ITListenSyslogGroovy.groovy}}) but the class name inside was not changed. I 
> believe the maven filter is on the class name, not the file name, because 
> this test still runs in normal builds. It breaks inconsistently on my 
> machine, so I renamed the class to ensure it is not run as part of the usual 
> unit test battery. This is required for 1.7.1 to be released. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi pull request #2883: NIFI-5415 Renamed ListenSyslogGroovyTest to ITListe...

2018-07-13 Thread asfgit
Github user asfgit closed the pull request at:

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


---


[jira] [Updated] (NIFI-5415) Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy

2018-07-13 Thread Matt Burgess (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5415?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Burgess updated NIFI-5415:
---
Fix Version/s: 1.8.0

> Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy
> --
>
> Key: NIFI-5415
> URL: https://issues.apache.org/jira/browse/NIFI-5415
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Tools and Build
>Affects Versions: 1.7.0
>Reporter: Andy LoPresto
>Assignee: Andy LoPresto
>Priority: Blocker
>  Labels: groovy, integration-test, maven, syslog, test
> Fix For: 1.8.0, 1.7.1
>
>
> [~joewitt] made an effort to move inconsistent and network/timing-dependent 
> unit tests to integration tests via NIFI-4659, NIFI-4660, NIFI-4662, and 
> NIFI-4664. 
> This file was renamed (from {{ListenSyslogGroovyTest.groovy}} to 
> {{ITListenSyslogGroovy.groovy}}) but the class name inside was not changed. I 
> believe the maven filter is on the class name, not the file name, because 
> this test still runs in normal builds. It breaks inconsistently on my 
> machine, so I renamed the class to ensure it is not run as part of the usual 
> unit test battery. This is required for 1.7.1 to be released. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5415) Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543217#comment-16543217
 ] 

ASF GitHub Bot commented on NIFI-5415:
--

Github user mattyb149 commented on the issue:

https://github.com/apache/nifi/pull/2883
  
+1 LGTM, ran with and without the change, verified the test was still being 
run as a unit test without the change, and doesn't run as a unit test with the 
change. Thanks for the fix! Merging to master


> Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy
> --
>
> Key: NIFI-5415
> URL: https://issues.apache.org/jira/browse/NIFI-5415
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Tools and Build
>Affects Versions: 1.7.0
>Reporter: Andy LoPresto
>Assignee: Andy LoPresto
>Priority: Blocker
>  Labels: groovy, integration-test, maven, syslog, test
> Fix For: 1.8.0, 1.7.1
>
>
> [~joewitt] made an effort to move inconsistent and network/timing-dependent 
> unit tests to integration tests via NIFI-4659, NIFI-4660, NIFI-4662, and 
> NIFI-4664. 
> This file was renamed (from {{ListenSyslogGroovyTest.groovy}} to 
> {{ITListenSyslogGroovy.groovy}}) but the class name inside was not changed. I 
> believe the maven filter is on the class name, not the file name, because 
> this test still runs in normal builds. It breaks inconsistently on my 
> machine, so I renamed the class to ensure it is not run as part of the usual 
> unit test battery. This is required for 1.7.1 to be released. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-5415) Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy

2018-07-13 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543218#comment-16543218
 ] 

ASF subversion and git services commented on NIFI-5415:
---

Commit ffbff42421c285400f4425206490faba07036946 in nifi's branch 
refs/heads/master from [~alopresto]
[ https://git-wip-us.apache.org/repos/asf?p=nifi.git;h=ffbff42 ]

NIFI-5415 Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy.

Signed-off-by: Matthew Burgess 

This closes #2883


> Renamed ListenSyslogGroovyTest to ITListenSyslogGroovy
> --
>
> Key: NIFI-5415
> URL: https://issues.apache.org/jira/browse/NIFI-5415
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Tools and Build
>Affects Versions: 1.7.0
>Reporter: Andy LoPresto
>Assignee: Andy LoPresto
>Priority: Blocker
>  Labels: groovy, integration-test, maven, syslog, test
> Fix For: 1.8.0, 1.7.1
>
>
> [~joewitt] made an effort to move inconsistent and network/timing-dependent 
> unit tests to integration tests via NIFI-4659, NIFI-4660, NIFI-4662, and 
> NIFI-4664. 
> This file was renamed (from {{ListenSyslogGroovyTest.groovy}} to 
> {{ITListenSyslogGroovy.groovy}}) but the class name inside was not changed. I 
> believe the maven filter is on the class name, not the file name, because 
> this test still runs in normal builds. It breaks inconsistently on my 
> machine, so I renamed the class to ensure it is not run as part of the usual 
> unit test battery. This is required for 1.7.1 to be released. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi issue #2883: NIFI-5415 Renamed ListenSyslogGroovyTest to ITListenSyslog...

2018-07-13 Thread mattyb149
Github user mattyb149 commented on the issue:

https://github.com/apache/nifi/pull/2883
  
+1 LGTM, ran with and without the change, verified the test was still being 
run as a unit test without the change, and doesn't run as a unit test with the 
change. Thanks for the fix! Merging to master


---


[jira] [Commented] (NIFI-5426) Use NIO.2 API for ListFile to avoid multiple disk reads

2018-07-13 Thread Mark Payne (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543212#comment-16543212
 ] 

Mark Payne commented on NIFI-5426:
--

To clarify, the benefit here of NIO.2 is that when a listing is obtained, the 
Path objects are able (at least in many circumstances) to cache the basic file 
attributes. However, when using the File API, a listing contains only the list 
of filenames, and we then have to ask the File System again about the basic 
attributes such as "is this a directory?" "last mod time". While this is still 
fast in most scenarios, for a network-mounted drive, this can sometimes lead to 
many minutes to list just a couple thousand files.

> Use NIO.2 API for ListFile to avoid multiple disk reads
> ---
>
> Key: NIFI-5426
> URL: https://issues.apache.org/jira/browse/NIFI-5426
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Extensions
>Reporter: Mark Payne
>Priority: Major
>
> NIFI-4631 attempted to do something similar but had issues when running 
> against CIFS mounts, reportedly. However, I think the issue was not the use 
> necessarily of NIO.2 but the way in which it was used. It attempted to use 
> walkTree and did did not appear to follow symlinks. As a result, this caused 
> failures in some scenarios. We should look at using NIO.2 and probably obtain 
> a Stream.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (NIFI-5426) Use NIO.2 API for ListFile to avoid multiple disk reads

2018-07-13 Thread Mark Payne (JIRA)
Mark Payne created NIFI-5426:


 Summary: Use NIO.2 API for ListFile to avoid multiple disk reads
 Key: NIFI-5426
 URL: https://issues.apache.org/jira/browse/NIFI-5426
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Extensions
Reporter: Mark Payne


NIFI-4631 attempted to do something similar but had issues when running against 
CIFS mounts, reportedly. However, I think the issue was not the use necessarily 
of NIO.2 but the way in which it was used. It attempted to use walkTree and did 
did not appear to follow symlinks. As a result, this caused failures in some 
scenarios. We should look at using NIO.2 and probably obtain a Stream.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (NIFIREG-185) Add element id's for User testing

2018-07-13 Thread Scott Aslan (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFIREG-185?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Aslan resolved NIFIREG-185.
-
Resolution: Fixed

> Add element id's for User testing
> -
>
> Key: NIFIREG-185
> URL: https://issues.apache.org/jira/browse/NIFIREG-185
> Project: NiFi Registry
>  Issue Type: Task
>Reporter: Andrew Lim
>Assignee: Andrew Lim
>Priority: Minor
>
> I need some element id's added to perform some automated UI testing around 
> User test cases (User creation/deletion/duplicates etc.).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFIREG-185) Add element id's for User testing

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFIREG-185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543185#comment-16543185
 ] 

ASF GitHub Bot commented on NIFIREG-185:


Github user scottyaslan commented on the issue:

https://github.com/apache/nifi-registry/pull/130
  
Thanks @andrewmlim this has been merged to master


> Add element id's for User testing
> -
>
> Key: NIFIREG-185
> URL: https://issues.apache.org/jira/browse/NIFIREG-185
> Project: NiFi Registry
>  Issue Type: Task
>Reporter: Andrew Lim
>Assignee: Andrew Lim
>Priority: Minor
>
> I need some element id's added to perform some automated UI testing around 
> User test cases (User creation/deletion/duplicates etc.).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi-registry issue #130: NIFIREG-185 Added element id's to facilitate User ...

2018-07-13 Thread scottyaslan
Github user scottyaslan commented on the issue:

https://github.com/apache/nifi-registry/pull/130
  
Thanks @andrewmlim this has been merged to master


---


[GitHub] nifi-registry pull request #130: NIFIREG-185 Added element id's to facilitat...

2018-07-13 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/nifi-registry/pull/130


---


[jira] [Commented] (NIFI-5425) Remove dependency duplication of nifi-utils and nifi-processor-utils

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16543034#comment-16543034
 ] 

ASF GitHub Bot commented on NIFI-5425:
--

GitHub user HorizonNet opened a pull request:

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

NIFI-5425 Removed duplicated declaration of nifi-utils and 
nifi-processor-utils

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?
- [ ] 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/ultratendency/nifi NIFI-5425

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

https://github.com/apache/nifi/pull/2888.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 #2888


commit 2ac2c37afa169cf0272c70d51744481619f58582
Author: Jan Hentschel 
Date:   2018-07-13T13:05:30Z

NIFI-5425 Removed duplicated declaration of nifi-utils and 
nifi-processor-utils




> Remove dependency duplication of nifi-utils and nifi-processor-utils
> 
>
> Key: NIFI-5425
> URL: https://issues.apache.org/jira/browse/NIFI-5425
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Jan Hentschel
>Priority: Trivial
>
> The dependencies {{nifi-utils}} and {{nifi-processor-utils}} are declared 
> twice (through inheritance) in some modules. The declarations in the 
> sub-modules can be removed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi pull request #2888: NIFI-5425 Removed duplicated declaration of nifi-ut...

2018-07-13 Thread HorizonNet
GitHub user HorizonNet opened a pull request:

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

NIFI-5425 Removed duplicated declaration of nifi-utils and 
nifi-processor-utils

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?
- [ ] 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/ultratendency/nifi NIFI-5425

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

https://github.com/apache/nifi/pull/2888.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 #2888


commit 2ac2c37afa169cf0272c70d51744481619f58582
Author: Jan Hentschel 
Date:   2018-07-13T13:05:30Z

NIFI-5425 Removed duplicated declaration of nifi-utils and 
nifi-processor-utils




---


[jira] [Created] (NIFI-5425) Remove dependency duplication of nifi-utils and nifi-processor-utils

2018-07-13 Thread Jan Hentschel (JIRA)
Jan Hentschel created NIFI-5425:
---

 Summary: Remove dependency duplication of nifi-utils and 
nifi-processor-utils
 Key: NIFI-5425
 URL: https://issues.apache.org/jira/browse/NIFI-5425
 Project: Apache NiFi
  Issue Type: Improvement
Reporter: Jan Hentschel


The dependencies {{nifi-utils}} and {{nifi-processor-utils}} are declared twice 
(through inheritance) in some modules. The declarations in the sub-modules can 
be removed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (NIFI-5424) Addition of a dark mode

2018-07-13 Thread Richard Midwinter (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Richard Midwinter updated NIFI-5424:

Description: 
I've got a partial dark theme available for NiFi.

Screenshot attached.

Known missing items in rough order of severity -
 * There are various iframes in dialog boxes, none of these inherit any styling.
 * Needs review to see if there's a better way of including it (and potentially 
other themes)
 * The styling could do with applying, or improving in minor areas; dialog 
close buttons, sliders.

Work in progress is here:

https://github.com/richmidwinter/nifi/commits/nifi-5424

  was:
I've got a partial dark theme available for NiFi.

Screenshot attached.

Known missing items in rough order of severity -
 * There are various iframes in dialog boxes, none of these inherit any styling.
 * Needs review to see if there's a better way of including it (and potentially 
other themes)
 * The styling could do with applying, or improving in minor areas; dialog 
close buttons, sliders.


> Addition of a dark mode
> ---
>
> Key: NIFI-5424
> URL: https://issues.apache.org/jira/browse/NIFI-5424
> Project: Apache NiFi
>  Issue Type: New Feature
>  Components: Core UI
>Affects Versions: 1.7.0
>Reporter: Richard Midwinter
>Priority: Major
> Attachments: lSofSq5.jpg
>
>
> I've got a partial dark theme available for NiFi.
> Screenshot attached.
> Known missing items in rough order of severity -
>  * There are various iframes in dialog boxes, none of these inherit any 
> styling.
>  * Needs review to see if there's a better way of including it (and 
> potentially other themes)
>  * The styling could do with applying, or improving in minor areas; dialog 
> close buttons, sliders.
> Work in progress is here:
> https://github.com/richmidwinter/nifi/commits/nifi-5424



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (NIFI-5424) Addition of a dark mode

2018-07-13 Thread Richard Midwinter (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Richard Midwinter updated NIFI-5424:

Priority: Minor  (was: Major)

> Addition of a dark mode
> ---
>
> Key: NIFI-5424
> URL: https://issues.apache.org/jira/browse/NIFI-5424
> Project: Apache NiFi
>  Issue Type: New Feature
>  Components: Core UI
>Affects Versions: 1.7.0
>Reporter: Richard Midwinter
>Priority: Minor
> Attachments: lSofSq5.jpg
>
>
> I've got a partial dark theme available for NiFi.
> Screenshot attached.
> Known missing items in rough order of severity -
>  * There are various iframes in dialog boxes, none of these inherit any 
> styling.
>  * Needs review to see if there's a better way of including it (and 
> potentially other themes)
>  * The styling could do with applying, or improving in minor areas; dialog 
> close buttons, sliders.
> Work in progress is here:
> https://github.com/richmidwinter/nifi/commits/nifi-5424



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (NIFI-5424) Addition of a dark mode

2018-07-13 Thread Richard Midwinter (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Richard Midwinter updated NIFI-5424:

Summary: Addition of a dark mode  (was: Add dark mode)

> Addition of a dark mode
> ---
>
> Key: NIFI-5424
> URL: https://issues.apache.org/jira/browse/NIFI-5424
> Project: Apache NiFi
>  Issue Type: New Feature
>  Components: Core UI
>Affects Versions: 1.7.0
>Reporter: Richard Midwinter
>Priority: Major
> Attachments: lSofSq5.jpg
>
>
> I've got a partial dark theme available for NiFi.
> Screenshot attached.
> Known missing items in rough order of severity -
>  * There are various iframes in dialog boxes, none of these inherit any 
> styling.
>  * Needs review to see if there's a better way of including it (and 
> potentially other themes)
>  * The styling could do with applying, or improving in minor areas; dialog 
> close buttons, sliders.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (NIFI-5424) Add dark mode

2018-07-13 Thread Richard Midwinter (JIRA)
Richard Midwinter created NIFI-5424:
---

 Summary: Add dark mode
 Key: NIFI-5424
 URL: https://issues.apache.org/jira/browse/NIFI-5424
 Project: Apache NiFi
  Issue Type: New Feature
  Components: Core UI
Affects Versions: 1.7.0
Reporter: Richard Midwinter
 Attachments: lSofSq5.jpg

I've got a partial dark theme available for NiFi.

Screenshot attached.

Known missing items in rough order of severity -
 * There are various iframes in dialog boxes, none of these inherit any styling.
 * Needs review to see if there's a better way of including it (and potentially 
other themes)
 * The styling could do with applying, or improving in minor areas; dialog 
close buttons, sliders.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NIFI-3993) Upgrade embedded ZooKeeper version

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-3993?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16542949#comment-16542949
 ] 

ASF GitHub Bot commented on NIFI-3993:
--

Github user joewitt commented on the issue:

https://github.com/apache/nifi/pull/2884
  
@jtstorck  @YolandaMDavis  and others interested - i recall some comments 
about holding off on updating zookeeper because of compatibility, etc..  Is 
that right?

@HorizonNet did you test the migrator and cluster functions with these 
changes including secure/unsecure settings by chance?


> Upgrade embedded ZooKeeper version
> --
>
> Key: NIFI-3993
> URL: https://issues.apache.org/jira/browse/NIFI-3993
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Core Framework
>Affects Versions: 1.2.0
>Reporter: Mark Bean
>Priority: Major
>
> In a Cluster configuration, Nodes are periodically disconnected from the 
> Cluster, and then reconnected. These events correspond to the following error:
> ERROR [CommitProcessor:1] o.apache.zookeeper.server.NIOServerCnxn Unexpected 
> Exception:
> java.nio.channels.CancelledKeyException: null
> at sun.nio.ch.SelectionKeyImpl.ensureValid(SectionKeyImpl.java:73)
> at sun.nio.ch.SelectionKeyImpl.interestOps(SelctionKeyImpl.java:77)
> at 
> org.apache.zookeeper.server.NIOServerCnxn.sendBuffer(NIOServerCnxn.java:151)
> at 
> org.apache.zookeeper.server.NIOServerCnXn.sendResopnse(NIOServerCnxn.java:1081)
> at 
> org.apache.zookeeper.server.FinalRequestProcessor.processRequest(FinalRequestProcessor.java:404)
> at 
> org.apache.zookeeper.server.quorum.CommitProcessor.run(CommitProcessor.java:74)
> This error was reported in ZooKeeper JIRA [1], and reported as fixed in 
> version 3.4.10, the current stable build. As additional confirmation, when 
> using a stand-alone ZK, 3.4.10, rather than the embedded ZK, the above error 
> was no longer observed.
> Update NiFi to use ZK 3.4.10
> [1] https://issues.apache.org/jira/browse/ZOOKEEPER-2044



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi issue #2884: NIFI-3993 Updated the ZooKeeper version to 3.4.10

2018-07-13 Thread joewitt
Github user joewitt commented on the issue:

https://github.com/apache/nifi/pull/2884
  
@jtstorck  @YolandaMDavis  and others interested - i recall some comments 
about holding off on updating zookeeper because of compatibility, etc..  Is 
that right?

@HorizonNet did you test the migrator and cluster functions with these 
changes including secure/unsecure settings by chance?


---


[jira] [Commented] (NIFI-5423) junit and slf4j-simple dependencies duplicated in sub-modules

2018-07-13 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/NIFI-5423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16542947#comment-16542947
 ] 

ASF GitHub Bot commented on NIFI-5423:
--

GitHub user HorizonNet opened a pull request:

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

NIFI-5423 Removed duplicated dependencies in sub-modules

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?
- [ ] 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/ultratendency/nifi NIFI-5423

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

https://github.com/apache/nifi/pull/2887.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 #2887


commit d0a710d1630d8bdca643d770b8ee8c768c089eda
Author: Jan Hentschel 
Date:   2018-07-13T11:51:14Z

NIFI-5423 Removed duplicated dependencies in sub-modules




> junit and slf4j-simple dependencies duplicated in sub-modules
> -
>
> Key: NIFI-5423
> URL: https://issues.apache.org/jira/browse/NIFI-5423
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Jan Hentschel
>Priority: Minor
>
> {{junit}} and {{slf4j-simple}} are declared in the root POM, but some 
> sub-modules duplicate them in their definition. The duplicated definitions in 
> the sub-modules should be removed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] nifi pull request #2887: NIFI-5423 Removed duplicated dependencies in sub-mo...

2018-07-13 Thread HorizonNet
GitHub user HorizonNet opened a pull request:

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

NIFI-5423 Removed duplicated dependencies in sub-modules

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?
- [ ] 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/ultratendency/nifi NIFI-5423

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

https://github.com/apache/nifi/pull/2887.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 #2887


commit d0a710d1630d8bdca643d770b8ee8c768c089eda
Author: Jan Hentschel 
Date:   2018-07-13T11:51:14Z

NIFI-5423 Removed duplicated dependencies in sub-modules




---


[jira] [Created] (NIFI-5423) junit and slf4j-simple dependencies duplicated in sub-modules

2018-07-13 Thread Jan Hentschel (JIRA)
Jan Hentschel created NIFI-5423:
---

 Summary: junit and slf4j-simple dependencies duplicated in 
sub-modules
 Key: NIFI-5423
 URL: https://issues.apache.org/jira/browse/NIFI-5423
 Project: Apache NiFi
  Issue Type: Improvement
Reporter: Jan Hentschel


{{junit}} and {{slf4j-simple}} are declared in the root POM, but some 
sub-modules duplicate them in their definition. The duplicated definitions in 
the sub-modules should be removed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (NIFI-5422) Make ExecuteScript thread safe & use CompiledScript where possible

2018-07-13 Thread Ramon Havermans (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5422?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ramon Havermans updated NIFI-5422:
--
Description: 
In ExcuteScript Bindings are reused, which makes it non thread safe. Most 
engines can use a Compiled version of the engine, which makes it faster.

Pull request will follow.

  was:In ExcuteScript Bindings are reused, which makes it non thread safe. Most 
engines can use a Compiled version of the engine, which makes it faster.


> Make ExecuteScript thread safe & use CompiledScript where possible
> --
>
> Key: NIFI-5422
> URL: https://issues.apache.org/jira/browse/NIFI-5422
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Core Framework
>Reporter: Ramon Havermans
>Priority: Major
>
> In ExcuteScript Bindings are reused, which makes it non thread safe. Most 
> engines can use a Compiled version of the engine, which makes it faster.
> Pull request will follow.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (NIFI-5422) Make ExecuteScript thread safe & use CompiledScript where possible

2018-07-13 Thread Ramon Havermans (JIRA)
Ramon Havermans created NIFI-5422:
-

 Summary: Make ExecuteScript thread safe & use CompiledScript where 
possible
 Key: NIFI-5422
 URL: https://issues.apache.org/jira/browse/NIFI-5422
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Core Framework
Reporter: Ramon Havermans


In ExcuteScript Bindings are reused, which makes it non thread safe. Most 
engines can use a Compiled version of the engine, which makes it faster.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (NIFI-5421) Provide support for Kerberized zookeeper in docker instances

2018-07-13 Thread Sarthak (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5421?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sarthak updated NIFI-5421:
--
Description: We should make the Docker image configurable to support 
Kerberized zookeeper in NiFi  (was: We should make the Docker image 
configurable to support all of the common security options available in NiFi)

> Provide support for Kerberized zookeeper in docker instances
> 
>
> Key: NIFI-5421
> URL: https://issues.apache.org/jira/browse/NIFI-5421
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Docker
>Affects Versions: 1.7.0
>Reporter: Sarthak
>Priority: Major
>
> We should make the Docker image configurable to support Kerberized zookeeper 
> in NiFi



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Assigned] (NIFI-5421) Provide support for Kerberized zookeeper in docker instances

2018-07-13 Thread Sarthak (JIRA)


 [ 
https://issues.apache.org/jira/browse/NIFI-5421?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sarthak reassigned NIFI-5421:
-

Assignee: Sarthak

> Provide support for Kerberized zookeeper in docker instances
> 
>
> Key: NIFI-5421
> URL: https://issues.apache.org/jira/browse/NIFI-5421
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Docker
>Affects Versions: 1.7.0
>Reporter: Sarthak
>Assignee: Sarthak
>Priority: Major
>
> We should make the Docker image configurable to support Kerberized zookeeper 
> in NiFi



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)