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

Added support for a string literal arguments to functions, which are pas... #632

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

PeterHatch
Copy link

...sed through to Solr.

@PeterHatch
Copy link
Author

This addresses issue #592, and also is a more general (and less able to reject improper values) version of pull request #572.

@PeterHatch
Copy link
Author

Oh, and the example added (and the function I and the author of issue #592 wanted to work) is directly from http://wiki.apache.org/solr/SolrRelevancyFAQ#How_can_I_boost_the_score_of_newer_documents

hybridindie added a commit to hybridindie/sunspot that referenced this pull request Mar 1, 2015
@njakobsen
Copy link
Contributor

I'll be able to merge this if you provide a test to verify the behaviour instead of just removing the test that expects it to explode.

@PeterHatch
Copy link
Author

Okay, I added a simple test. Let me know if there's more you'd like done.

@PeterHatch
Copy link
Author

Just rebased this to be based off the latest version. (And mistakes while doing so meant this was closed for a bit due to my branch being identical to the main branch.)

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.

None yet

2 participants