flux icon indicating copy to clipboard operation
flux copied to clipboard

Dev - changed a lot, please double check!

Open JanDornbusch opened this issue 5 years ago • 2 comments

Sorry, last time I did not test against real Solr as already reported in #20. Changed to dev as this is a bigger change and I would like a 2nd person to watch over before move it to be master.

This one includes few other PRs: #18 #17 #16 #12 as I was changing mostly everything I've included and tested their ideas too.

The new tests will require a zk with solr running and will test basic functionality against them to be sure everything works fine! I'm sure a few more tests could be added maybe #14 could be an inspiration about this.

JanDornbusch avatar Jul 11 '20 13:07 JanDornbusch

This one includes few other PRs: #18 #17 #16 #12 as I was changing mostly everything I've included and tested their ideas too. Awesome, thank you!

mwmitchell avatar Jul 12 '20 02:07 mwmitchell

Currently playing with Solr. Who knows what else I will find. As said might be a good way to leave it in dev branch some more time to see if everything works as expected!

JanDornbusch avatar Jul 12 '20 09:07 JanDornbusch