You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Oliver Szabo <os...@hortonworks.com> on 2017/04/24 19:07:53 UTC

Review Request 58679: Fix AmbariSolrCloudCLI package name in solr-client script

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58679/
-----------------------------------------------------------

Review request for Ambari, Miklos Gergely, Robert Nettleton, Sandor Magyari, and Sebastian Toader.


Bugs: AMBARI-20837
    https://issues.apache.org/jira/browse/AMBARI-20837


Repository: ambari


Description
-------

fix package name for solr-client cli script, as the logsearch package does not exist


Diffs
-----

  ambari-infra/ambari-infra-solr-client/src/main/resources/solrCloudCli.sh cd47f06 


Diff: https://reviews.apache.org/r/58679/diff/1/


Testing
-------

done


Thanks,

Oliver Szabo


Re: Review Request 58679: Fix AmbariSolrCloudCLI package name in solr-client script

Posted by Sebastian Toader <st...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58679/#review172900
-----------------------------------------------------------


Ship it!




Ship It!

- Sebastian Toader


On April 24, 2017, 9:07 p.m., Oliver Szabo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58679/
> -----------------------------------------------------------
> 
> (Updated April 24, 2017, 9:07 p.m.)
> 
> 
> Review request for Ambari, Miklos Gergely, Robert Nettleton, Sandor Magyari, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-20837
>     https://issues.apache.org/jira/browse/AMBARI-20837
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> fix package name for solr-client cli script, as the logsearch package does not exist
> 
> 
> Diffs
> -----
> 
>   ambari-infra/ambari-infra-solr-client/src/main/resources/solrCloudCli.sh cd47f06 
> 
> 
> Diff: https://reviews.apache.org/r/58679/diff/1/
> 
> 
> Testing
> -------
> 
> done
> 
> 
> Thanks,
> 
> Oliver Szabo
> 
>


Re: Review Request 58679: Fix AmbariSolrCloudCLI package name in solr-client script

Posted by Robert Nettleton <rn...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58679/#review172839
-----------------------------------------------------------


Ship it!




Ship It!

- Robert Nettleton


On April 24, 2017, 7:07 p.m., Oliver Szabo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58679/
> -----------------------------------------------------------
> 
> (Updated April 24, 2017, 7:07 p.m.)
> 
> 
> Review request for Ambari, Miklos Gergely, Robert Nettleton, Sandor Magyari, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-20837
>     https://issues.apache.org/jira/browse/AMBARI-20837
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> fix package name for solr-client cli script, as the logsearch package does not exist
> 
> 
> Diffs
> -----
> 
>   ambari-infra/ambari-infra-solr-client/src/main/resources/solrCloudCli.sh cd47f06 
> 
> 
> Diff: https://reviews.apache.org/r/58679/diff/1/
> 
> 
> Testing
> -------
> 
> done
> 
> 
> Thanks,
> 
> Oliver Szabo
> 
>


Re: Review Request 58679: Fix AmbariSolrCloudCLI package name in solr-client script

Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58679/#review172837
-----------------------------------------------------------


Ship it!




Ship It!

- Alejandro Fernandez


On April 24, 2017, 7:07 p.m., Oliver Szabo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58679/
> -----------------------------------------------------------
> 
> (Updated April 24, 2017, 7:07 p.m.)
> 
> 
> Review request for Ambari, Miklos Gergely, Robert Nettleton, Sandor Magyari, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-20837
>     https://issues.apache.org/jira/browse/AMBARI-20837
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> fix package name for solr-client cli script, as the logsearch package does not exist
> 
> 
> Diffs
> -----
> 
>   ambari-infra/ambari-infra-solr-client/src/main/resources/solrCloudCli.sh cd47f06 
> 
> 
> Diff: https://reviews.apache.org/r/58679/diff/1/
> 
> 
> Testing
> -------
> 
> done
> 
> 
> Thanks,
> 
> Oliver Szabo
> 
>


Re: Review Request 58679: Fix AmbariSolrCloudCLI package name in solr-client script

Posted by Miklos Gergely <mg...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58679/#review172842
-----------------------------------------------------------


Ship it!




Ship It!

- Miklos Gergely


On April 24, 2017, 7:07 p.m., Oliver Szabo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58679/
> -----------------------------------------------------------
> 
> (Updated April 24, 2017, 7:07 p.m.)
> 
> 
> Review request for Ambari, Miklos Gergely, Robert Nettleton, Sandor Magyari, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-20837
>     https://issues.apache.org/jira/browse/AMBARI-20837
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> fix package name for solr-client cli script, as the logsearch package does not exist
> 
> 
> Diffs
> -----
> 
>   ambari-infra/ambari-infra-solr-client/src/main/resources/solrCloudCli.sh cd47f06 
> 
> 
> Diff: https://reviews.apache.org/r/58679/diff/1/
> 
> 
> Testing
> -------
> 
> done
> 
> 
> Thanks,
> 
> Oliver Szabo
> 
>