Houston Putman
Houston Putman
Purely responding to the URP response part, it’s definitely not possible for URP to send non-error responses. I do think its something we should implement though, since it will expand...
Hey @risdenk should we still try and get this in?
Completely agree with both of you that not forking would be the best path forward. It's hard company-policy-wise for me to move forward with either of those options, but I...
Ok I merged `main` in and got stuff working. Haven't really tested everything, but I did test that the packaging works, and the the `local_packages.json` and the `manifest.json` files get...
@sgift this looks clean and simple! Thanks for the contribution 🙂
@laminelam I'm not sure what the `merge` commit is, but you need to remove it and re-do the merge, like the second merge commit. Otherwise we can't tell which files...
The failing tests are threads leaking in `TestCancellableCollector` and `OrderedExecutorTest`. I can't reproduce the failures locally, but it seems like a harmless addition to try and kill the threads after...
Closing this as there was better work done to fix these.
Thanks for getting this started again @haythemkh! As a first-pass I suggest that we put the current solrj functionality into `solrj/core`. Once that is committed, we can then do what...
Ok I will do it following this getting merged