search

SwiftPackageIndex / SwiftPackageIndex-Server

The Swift Package Index is the place to find Swift packages!
https://swiftpackageindex.com
Apache License 2.0
522 stars 37 forks source link

removes the search logging #3142

Closed heckj closed 3 weeks ago

heckj commented 4 weeks ago

Removes the search logger from the dev infrastructure. Should improve the query response time for search in particular (at least in Dev) and reduce logging consumption in that environment as well.

daveverwer commented 3 weeks ago

This has the old problem of not running the tests because they are not under our repository. It works on other repos I am on, but has never worked here and I don't know how to fix it.

I'll re-make this PR under this repo and cherry pick these commits.

heckj commented 3 weeks ago

Well that's annoying - sorry gents, I didn't mean to make this annoying.

daveverwer commented 3 weeks ago

It was no problem!

finestructure commented 3 weeks ago

FYI, instead of cherry-picking you can also just push the branch into the SPI repo

heckj commented 3 weeks ago

I had a copy/fork of the repo (heckj/SwiftPackageIndex) from ages ago - was still working from that, but that path I guess is what the tests don't care for. Next time I'll make a branch directly on this org's repo and work from there.