[GitHub] nifi pull request: NIFI-988: PutDistributedMapCache processor

2015-10-20 Thread joemeszaros
Github user joemeszaros commented on the pull request: https://github.com/apache/nifi/pull/92#issuecomment-149458834 @markap14 Do you have any concern with this new processor? It is still open for a while and it would be grateful, if we could finish it by merging or closing the PR.

Configuration Driven Property Updates

2015-10-20 Thread Michael D. Coon
All,    I'm building a set of NiFi processors that allow data flow managers to dynamically configure reusable data processing modules wrapped inside NiFi processors. Imagine that we might want to use NiFi in some cases and Storm or Spark in others. The business logic in the processing modules

[GitHub] nifi pull request: [NIFI-987] Added Processor For Writing Events t...

2015-10-20 Thread busbey
Github user busbey commented on the pull request: https://github.com/apache/nifi/pull/91#issuecomment-149670303 Please edit commit message to start with JIRA key and brief summary. i.e.: ```commit NIFI-987 Add Reimann monitoring - Introduced nifi-riemann-bundle

[GitHub] nifi pull request: [NIFI-987] Added Processor For Writing Events t...

2015-10-20 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/91#issuecomment-149677321 just speculation, but if you have a bunch of metrics in a file from somewhere and want to post it to Riemann, a processor would be useful. Granted, when I think of people

Re: Newbie Dev Contribution : Questions

2015-10-20 Thread Sean Busbey
Oleg, Could we convince you to contribute a version of the IDE integration stuff that works within the current codebase? On Fri, Oct 16, 2015 at 5:39 AM, Venkatesh Sellappa wrote: > Thanks a bunch. > This is very good. > > I think one of the things that threw me off-course

[GitHub] nifi pull request: [NIFI-987] Added Processor For Writing Events t...

2015-10-20 Thread busbey
Github user busbey commented on the pull request: https://github.com/apache/nifi/pull/91#issuecomment-149672053 Why do this as a processor rather than as a reporting task? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] nifi pull request: [NIFI-987] Added Processor For Writing Events t...

2015-10-20 Thread rickysaltzer
Github user rickysaltzer commented on the pull request: https://github.com/apache/nifi/pull/91#issuecomment-149616531 Hey Joe - Not a problem on the delay! I'll take a look when I get a chance. Thanks! Thanks, Ricky --- If your project is set up for it, you can

[GitHub] nifi pull request: NIFI 944: Support tab & special delimiter chars...

2015-10-20 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/87#issuecomment-149617912 @joemeszaros - I'm not sure I disagree with you... however I was a bit troubled by the \n case when testing, and unicode is full of control characters (e.g. LRM (U+200E))

[GitHub] nifi pull request: NIFI-1021 added support for Provenance event st...

2015-10-20 Thread olegz
Github user olegz commented on the pull request: https://github.com/apache/nifi/pull/102#issuecomment-149621632 Closing after discussing it with Joe. Will resubmit once proposed implementation is realized within the context of the ReportingTask --- If your project is set up for it,

[GitHub] nifi pull request: NIFI-1021 added support for Provenance event st...

2015-10-20 Thread olegz
Github user olegz closed the pull request at: https://github.com/apache/nifi/pull/102 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] nifi pull request: [NIFI-987] Added Processor For Writing Events t...

2015-10-20 Thread rickysaltzer
Github user rickysaltzer commented on the pull request: https://github.com/apache/nifi/pull/91#issuecomment-149631055 The conflict was inside of the `pom.xml` inside the `nifi-nar-bundles` module. The new couchbase module was conflicting with the new riemann module inside the ``

A few questions on tracking project participation

2015-10-20 Thread Sean Busbey
When a jira is completed by a non-committer, who does the jira get assigned to? I couldn't find a relevant section in the contributor guide (maybe we need a "for committers" section?). If I wanted to get a sense of project activity (across committers and non-committers alike), how would I get

Re: A few questions on tracking project participation

2015-10-20 Thread Sean Busbey
well that's frustrating. It was approximately: [1]: The use of ammending-author started as a way of handling gnarly cherry-picks back to earlier release lines in HBase, but I've seen it show up on a couple of multiple-author patches generally. The HBase reference guide is 404 at the moment, so

Re: Newbie Dev Contribution : Questions

2015-10-20 Thread Oleg Zhurakousky
Sure, I am ok with adding to contributor guide. Let me know how should I proceed with it (e.g. PR or a paragraph via this thread so someone can add it to the guide) Cheers Oleg Sent from my iPhone > On Oct 20, 2015, at 13:07, Sean Busbey wrote: > > I was thinking the

Re: A few questions on tracking project participation

2015-10-20 Thread Tony Kurc
Sean, Can you better define "completed by a non-committer"? Some cases to consider Is "a jira" complete when a patch is submitted or when it is merged into the code repository? If a patch requires a bit of work to integrate (and test), where does that fall? What if a patch provider doesn't have a

Re: Newbie Dev Contribution : Questions

2015-10-20 Thread Sean Busbey
A PR would be grand. Thanks in advance! On Tue, Oct 20, 2015 at 3:54 PM, Oleg Zhurakousky wrote: > Sure, I am ok with adding to contributor guide. Let me know how should I > proceed with it (e.g. PR or a paragraph via this thread so someone can add it > to the

Re: Newbie Dev Contribution : Questions

2015-10-20 Thread Oleg Zhurakousky
Sure, although I am not sure how it would manifest itself since this is simply a set of instructions which could go to docs/wiki etc. Is that what you meant? Cheers Oleg > On Oct 20, 2015, at 3:20 PM, Sean Busbey wrote: > > Oleg, > > Could we convince you to contribute a

Re: A few questions on tracking project participation

2015-10-20 Thread Tony Kurc
Sean, I think your footnote link got chopped. Tony On Tue, Oct 20, 2015 at 4:39 PM, Sean Busbey wrote: > Well, I admit that my questions were as much to get more questions as > to advance an idea of answers. :) > > > On Tue, Oct 20, 2015 at 3:16 PM, Tony Kurc

Re: Newbie Dev Contribution : Questions

2015-10-20 Thread Sean Busbey
I was thinking the contributor guide, but anywhere we can add a URL to is fine by me. :) On Tue, Oct 20, 2015 at 2:40 PM, Oleg Zhurakousky wrote: > Sure, although I am not sure how it would manifest itself since this is > simply a set of instructions which could go

Re: A few questions on tracking project participation

2015-10-20 Thread Sean Busbey
Well, I admit that my questions were as much to get more questions as to advance an idea of answers. :) On Tue, Oct 20, 2015 at 3:16 PM, Tony Kurc wrote: > Sean, > Can you better define "completed by a non-committer"? > Where the majority of the work is done by a

[GitHub] nifi pull request: NIFI 944: Support tab & special delimiter chars...

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

[GitHub] nifi pull request: NIFI 944: Support tab & special delimiter chars...

2015-10-20 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/87#issuecomment-149767800 I wrote a test which exhausts the unicode code points. \u is the most problematic as it threw exceptions ``` java.lang.UnsupportedOperationException: The

[GitHub] nifi pull request: NIFI 944: Support tab & special delimiter chars...

2015-10-20 Thread trkurc
Github user trkurc commented on the pull request: https://github.com/apache/nifi/pull/87#issuecomment-149768870 @joemeszaros I am going to open a bug ticket which sanity checks across the properties. I don't believe your pr exacerbates the issue, so should not be blocked by it. ---