-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[solr] Added tests for Solr binding #583
Conversation
- Default value is `false` and therefore expects solr to be running in stand-alone mode. | ||
|
||
- `solr.base.url` | ||
- The base URL in which to interface with a running Solr instance in Stand=alone mode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: Stand=alone
Dammit, did I comment on the wrong PR? Assume all my comments go to the other one then, if so. Sorry. :( |
19c1a1c
to
9f36bcc
Compare
@busbey addressed comments about putting reference to Solr config files and configuring log4j to go to stderr. |
limitations under the License. | ||
--> | ||
<!-- | ||
Copied from Apache Solr solr/solrj/src/test-files/solrj/solr/collection1/conf/schema.xml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a release tag (or lacking one a commit hash) would be preferable, so that folks who want to inspect the original can.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. Added to schema.xml and solrconfig.xml
[solr] Added tests for Solr binding
thanks! |
[solr] Added tests for Solr binding
[solr] Added tests for Solr binding
Relies on PR #574. Relates to @busbey comment about testing the Solr binding #574 (comment). Will most likely need to be rebased after #574 is merged.