You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by bbende <gi...@git.apache.org> on 2016/12/16 21:30:34 UTC

[GitHub] nifi pull request #1338: NIFI-3220 Add support for basic auth to Solr proces...

GitHub user bbende opened a pull request:

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

    NIFI-3220 Add support for basic auth to Solr processors

    Also made the Solr Location support expression location so it can be paramterized with the variable registry.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/bbende/nifi NIFI-3220

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

    https://github.com/apache/nifi/pull/1338.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 #1338
    
----
commit 656d47d6c9e1f972a582a1c71cf8547ed3910145
Author: Bryan Bende <bb...@apache.org>
Date:   2016-12-16T21:28:10Z

    NIFI-3220 Add support for basic auth to Solr processors

----


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] nifi pull request #1338: NIFI-3220 Add support for basic auth to Solr proces...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] nifi issue #1338: NIFI-3220 Add support for basic auth to Solr processors

Posted by bbende <gi...@git.apache.org>.
Github user bbende commented on the issue:

    https://github.com/apache/nifi/pull/1338
  
    @ijokarumawak yes we should definitely create another JIRA to convert GetSolr to use state management, I keep forgetting that it doesn't do that already.
    
    If you need any help with the basic auth setup let me know, I just followed Solr's wiki page:
    https://cwiki.apache.org/confluence/display/solr/Basic+Authentication+Plugin


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] nifi issue #1338: NIFI-3220 Add support for basic auth to Solr processors

Posted by ijokarumawak <gi...@git.apache.org>.
Github user ijokarumawak commented on the issue:

    https://github.com/apache/nifi/pull/1338
  
    Start reviewing, and testing with Solr..
    
    @bbende 
    This is an out of scope question, but let me ask..
    Should GetSolr utilize NiFi state management to store last query timestamp, so that other node can takeover if the node goes down? (I assume this processor is designed to run on a primary node) Or to retain the state across NiFi restart? 
    If so, I think I will create a JIRA for that. 


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] nifi issue #1338: NIFI-3220 Add support for basic auth to Solr processors

Posted by ijokarumawak <gi...@git.apache.org>.
Github user ijokarumawak commented on the issue:

    https://github.com/apache/nifi/pull/1338
  
    Thanks @bbende I've created [NIFI-3252: Make use of State management in GetSolr](https://issues.apache.org/jira/browse/NIFI-3252).


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] nifi issue #1338: NIFI-3220 Add support for basic auth to Solr processors

Posted by ijokarumawak <gi...@git.apache.org>.
Github user ijokarumawak commented on the issue:

    https://github.com/apache/nifi/pull/1338
  
    Tested with Solr cluster with basic authentication, both GetSolr and PutSolrContentStream were able to communicate with the Solr cluster.
    Thanks @bbende , I've merged this into master!


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---