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

Add methods removed attributes #7

Closed
GZyl opened this issue Apr 16, 2014 · 5 comments
Closed

Add methods removed attributes #7

GZyl opened this issue Apr 16, 2014 · 5 comments
Assignees

Comments

@GZyl
Copy link

GZyl commented Apr 16, 2014

Attributes for "add"

  • allowDups
  • overwritePending
  • overwriteCommitted

are removed in Solr4.0 - use "overwrite"

@rjkip
Copy link
Contributor

rjkip commented Apr 16, 2014

I'll look into this asap.

@rjkip rjkip self-assigned this Apr 16, 2014
@rjkip
Copy link
Contributor

rjkip commented Apr 16, 2014

To preserve the current API, would you agree overwrite is true when allowDups is false and overwritePending and overwriteCommitted are true?

It would seem adding overwrite="true/false" to <add /> would solve this issue. It should break neither < 4.0, nor >= 4.0.

@GZyl
Copy link
Author

GZyl commented Apr 16, 2014

overwritePending and overwriteCommitted default are negation of allowDups in Solr <4.0.

In Solr >= 4.0
overwrite - default is "true", meaning newer documents will replace previously added documents with the same uniqueKey.

For me overwrite = !allowDups

It should be connected with Apache_Solr_Compatibility_Solr4CompatibilityLayer

@rjkip
Copy link
Contributor

rjkip commented Apr 17, 2014

Ok, overwrite=!allowDups.

I think we should move all XML generation into the comp layer and tag v2.0.0, as it would be a BC break. If you require these changes soon or if you feel up to it, you can send me a PR. Otherwise I may look into it this weekend.

@GZyl
Copy link
Author

GZyl commented Apr 17, 2014

My first PR ( #8 ) on github ;]
Tested on PHP 5.5.10 (PHPUnit 3.7.32) is OK.

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

No branches or pull requests

2 participants