Skip to content
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

Fix for #90 - introducing support for the SOLR-ExtractionHandler #109

Open
wants to merge 48 commits into
base: v0.3.x
Choose a base branch
from

Conversation

marc-portier
Copy link
Collaborator

Including tests and updated test-material.
Thx for considering this.

juzerali and others added 30 commits November 18, 2012 22:00
…is kind of file through the github web interface ever again ;)
…gestion in issue lbdremy#85

Merge branch 'v0.3.x' into v0.3.x-patch85 - resolving minor conflict in package.json
introduces test/config and overwrite possibility on CLI via figc
Readme explains how to use it.
… exists in the schema and adding the extra returned data to compare
Making it not-conflicting with the lbdremy#88 implementation by renaming get --> handleGet()
Adding the handlePost to complete the use-case behind this
Not returning self but rather the request since that seems the standard pattern in v0.3.x
Conflicts:
	test/core-query-test.js
	test/group-query-test.js
	test/mlt-query-test.js
marc-portier and others added 18 commits July 31, 2014 05:06
…ame (some version between 3.3 and 4.9) So making that optional to include!
Introducing json-bigint to properly handle the too-big-number issue in classic json parsing.

Adding the test-cases that show the need for this in regular solr fields of type long (_l suffix)
and the fact that the solr 4.x optimistic locking feature (ie the  _version_ field) needs this badly.

Caveat: json-bigint is currently more restrictive on json parsing then the classic JSON.parse
Namely: it renders an error on duplicate key fields in the json. Since the response-header of the ping
in solr introduces such a duplicate key this commit chooses to force omitHeader=true for the ping request.

This can be reconsidered if json-bigint would allow for a more lenient parse() in line with the classic one.
… in json-bigint we can re-introduce the headers in the ping response and return to the classic assertion scheme in the ping-tests.

This should nicely roundup issue lbdremy#71.
Usage:
var client = solr.createClient({ secure : true});
Manually handled Conflicts:
	lib/solr.js
	package.json
	test/bigint-test.js
…h the signature of createAddStream() - changing the test to use the new signature as sample.

Switch to using the https support recently introduced in v0.3.x
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants