HoustonPutman commented on pull request #120: URL: https://github.com/apache/solr/pull/120#issuecomment-896197689
@athrog & @gerlowskija , I've gotten the precommit to pass (wow this adds a lot of dependencies). The full tests also pass 🎉 As per my comment above, I believe there are 3 areas left to finish before this work can be merged. - [ ] Documentation of the S3 Repo option in the solr-ref-guide - [ ] The tests currently have a lot of asserts without comments, these should be easy to debug when tests fail. Please add messages to any asserts there. - [ ] Finish debugging the async error, if it still occurs. (I will certainly be testing this, so I'll take responsibility for this task) Once those are done, I will be happy to merge and backport this to 8x. Let me know what your bandwith is, and when you think you can get these done by 🙂 No rush, just want to be able to plan accordingly on my side. Also just realized that upgrading the jackson smile dependency broke tests, so looking into fixing those currently... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
