vanng822 / go-solr

Solr client in Go, core admin, add docs, update, delete, search and more
MIT License
69 stars 42 forks source link

What versions of solr are supported? #45

Closed DallanQ closed 6 years ago

DallanQ commented 6 years ago

I'd like to use this with Solr 7. Will I have any problems?

vanng822 commented 6 years ago

Hi @DallanQ Sorry I can not answer your question. However this package is really simple so I don't see any problem. Please try it out and write an answer here, thanks

DallanQ commented 6 years ago

Thanks!

DallanQ commented 6 years ago

Tested with solr 7 and it works great!

ironsteel commented 5 years ago

There is an issue with the Update function. The url concatenation adds an extra slash before the query parameters, which does not result in updating the document. Testing with solr 7.5.0

Pull request coming soon.

vanng822 commented 5 years ago

@ironsteel thank you for update. Happy with a PR, thanks

ironsteel commented 5 years ago

Sorry my mistake, I forgot to add the commit parameter to the request. Sorry for the misleading comment.