add hotness queries and materializer (#25) #127
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
a45f75d
to
8e9c764
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
8e9c764
to
3f5694a
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
2 times, most recently
from
08b1d7c
to
62dc1dc
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
2 times, most recently
from
40a6f88
to
2dd6967
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
2dd6967
to
6437ea8
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
6437ea8
to
d33f5b4
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
d33f5b4
to
3185591
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
3185591
to
d8cca76
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
d8cca76
to
f00b179
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
f00b179
to
755ccd6
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
755ccd6
to
dd21faa
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
dd21faa
to
c99b044
Compare
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
c99b044
to
44d85f5
Compare
also reformatted some sql
itstolf
force-pushed
the
itstolf/hot-mess
branch
from
44d85f5
to
c79e441
Compare
itstolf
changed the title
strideynet
reviewed
Aug 16, 2023
There was a problem hiding this comment.
So this strategy looks good - but I do wonder if scoring the posts in Go rather than SQL might give us more flexibility in adding new Algos - but I can equally see this sucking from a perf aspect. My gut feeling is to run with what you've done here and we can always move it to Go in future if we want to.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
also reformatted some sql