[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-11-27 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/2671 If you take a look here https://nifi.apache.org/docs/nifi-docs/html/administration-guide.html#core-properties-br and checkout the 'nifi.nar.library.directory' section you can see how someone could

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-11-27 Thread ryanjdew
Github user ryanjdew commented on the issue: https://github.com/apache/nifi/pull/2671 @joewitt I understand the concern. We would like to simplify the user experience of using NiFi with MarkLogic processors. We have a current model for creating and releasing NARs via a GitHub

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-11-27 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/2671 as a good example where we should have though it through more as well we have some cool work done by the InfluxDB folks. They're now wanting to improve it in https://github.com/apache/nifi/pull/2743

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-11-27 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/2671 @ryanjdew We have addressed Travis-CI related tests issues and the build has been stable now since. It could break again as people add timing dependent tests that behave wildly different on slow

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-11-27 Thread ryanjdew
Github user ryanjdew commented on the issue: https://github.com/apache/nifi/pull/2671 Following up, are there any other concerns with this PR? If needed, I can provide credentials to a MarkLogic instance for testing. ---

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-10-31 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/2671 team; given https://github.com/marklogic/nifi/releases could we consider closing this PR and keeping the MarkLogic artifact creation/maintenance something MarkLogic takes care of at this time? It is

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-09-11 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2671 @vivekmuniyandi you have a merge conflict now. ---

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-14 Thread vivekmuniyandi
Github user vivekmuniyandi commented on the issue: https://github.com/apache/nifi/pull/2671 @joewitt I have addressed all your comments except for the License and Notice comments. Can you please let us know what more should we add apart from the LICENSE and NOTICE file prepared by

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-14 Thread vivekmuniyandi
Github user vivekmuniyandi commented on the issue: https://github.com/apache/nifi/pull/2671 @MikeThomsen Sure, will add that to our backlog. Thanks! ---

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-14 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2671 @vivekmuniyandi Unrelated, but I don't have your email: consider adding a MarkLogicLookupService in a future sprint. You can look at HBaseLookupService and MongoDBLookupService as examples. Might

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-10 Thread vivekmuniyandi
Github user vivekmuniyandi commented on the issue: https://github.com/apache/nifi/pull/2671 Thanks @joewitt for the comment. I am addressing all the changes you have mentioned. I will address the SSLContext and remove the Kerberos and Certificate auth for now. ``` The other

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-09 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/2671 Ok i've attached a patch which helps with some aspects of POM construction, flagging things like resource utilization since it appears to be loading full content into memory, and renaming the service

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-07 Thread vivekmuniyandi
Github user vivekmuniyandi commented on the issue: https://github.com/apache/nifi/pull/2671 @MikeThomsen We don't have a public MarkLogic Docker image but we do have [this](https://hub.docker.com/r/patrickmcelwee/marklogic-dependencies/) on Docker Hub which would give you a head

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-07 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2671 @vivekmuniyandi I'll try to find time to set up a MarkLogic node for testing some time in the next few days (day job and such is getting in the way). In the mean time, I would suggest reaching

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-04 Thread vivekmuniyandi
Github user vivekmuniyandi commented on the issue: https://github.com/apache/nifi/pull/2671 Thanks @MikeThomsen for the comment. `PutMarkLogicRecord` is definitely on our roadmap but I am not sure when we will be able to get to it. We have an internal sprint for NiFi. We will add

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-04 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2671 BTW, I would strongly recommend your team discuss adding a `PutMarkLogicRecord` processor so you can do a bulk ingestion invent from a single flowfile. We have quite a few good implementations

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-04 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2671 @vivekmuniyandi I'll try to build a MarkLogic Docker image and share it on Docker Hub so others can use that if they want. ---

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-03 Thread vivekmuniyandi
Github user vivekmuniyandi commented on the issue: https://github.com/apache/nifi/pull/2671 Thanks @MikeThomsen ! Have made the changes. ---

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-03 Thread vivekmuniyandi
Github user vivekmuniyandi commented on the issue: https://github.com/apache/nifi/pull/2671 I followed this link - https://cwiki.apache.org/confluence/display/NIFI/Contributor+Guide#ContributorGuide-Keepingyourfeaturebranchcurrent and looks like this pulled other's commits as

[GitHub] nifi issue #2671: NiFi-5102 - Adding Processors for MarkLogic DB

2018-05-03 Thread MikeThomsen
Github user MikeThomsen commented on the issue: https://github.com/apache/nifi/pull/2671 @vivekmuniyandi looks like you pulled a few other folks' commits in with your last push. Do this to clear that up: 1. git checkout master 2. git pull master 3. git checkout