Some (probably inaccurate) stats about Dub packages

WebFreak001 d.forum at webfreak.org
Fri Oct 22 11:47:09 UTC 2021


On Friday, 22 October 2021 at 11:17:11 UTC, SealabJaster wrote:
> On Friday, 22 October 2021 at 07:07:01 UTC, Imperatorn wrote:
>> Maybe possible to integrate with 
>> https://github.com/dlang/dub-registry/pull/497
>
> I don't have any motivation to work on Dub, so someone else 
> would have to champion something like this through.

that PR is completely different and removes the internal MongoDB 
search, replacing it with "packageName.canFind(query)" - that is 
not the place you would want to extend this search into.

Additionally that PR fails to address problems that were brought 
up in review. It also mixes in the search changes (which I 
majorly disapprove of as they are right now) with bug fixes and 
deprecation fixes, which should really get into the code base but 
I won't merge in as long as they are included with the search 
changes.

As I have already commented in that PR it should really _extend_ 
the MongoDB text search, not replace it. Make it an aggregate 
query with a MongoDB $regex match (that makes it a simple string 
contains, escape regex characters with std.regex) that merges 
with the text search, giving the regex results higher text scores.

If you, the person reading this, are motivated in pushing through 
a search improvement in DUB, you can check that PR for hints 
where to start, but you should really create a new PR instead and 
let MongoDB do the work or introduce some other indexed search 
framework.


More information about the Digitalmars-d mailing list