Ignore clusterstate.json. Fixes #224.#315
Ignore clusterstate.json. Fixes #224.#315rokclimb15 wants to merge 2 commits intodjango-haystack:masterfrom rokclimb15:issue-224-unknown-collection
Conversation
|
Is there any upstream documentation about this being deprecated? We should probably have some sort of documentation / release notes link since I'm assuming there's at least some older version of Solr someone is running where this still works. |
|
I wasn't able to find any official documentation, likely because this is the internal API of Solr with ZooKeeper and is subject to change. The "official" way to do this would be to call the Solr HTTP API. I think it would make sense to do this in the context of a BC release and update the test suite to run Solr >= 7 (supported). I started to do that, but ran into a few snags with config files. |
Codecov Report
@@ Coverage Diff @@
## master #315 +/- ##
==========================================
- Coverage 94.35% 87.74% -6.61%
==========================================
Files 7 7
Lines 1328 1322 -6
==========================================
- Hits 1253 1160 -93
- Misses 75 162 +87
Continue to review full report at Codecov.
|
|
Closing in favor of #426 |
Remove clusterstate.json and listener since the node exists but Solr no longer updates it. This resulted in the collections property being overwritten with an empty state. Tested on Solr 7.7.2. Below is log output during a connection/index/search routine. Without clusterstate being watched the
ZooKeeperclass correctly connects tolive_nodes, pollscollections, and polls the collection state to get shards.I attempted to run the test suite, but it looks like the Solr package referenced is 404 in the test script. Additionally, Solr 4 is unsupported at this point I think.