Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added search indexing and querying #104

Merged
merged 10 commits into from
Aug 29, 2024
Merged

Added search indexing and querying #104

merged 10 commits into from
Aug 29, 2024

Conversation

Yantrio
Copy link
Member

@Yantrio Yantrio commented Aug 28, 2024

This PR adds 2 components

  • /search/pg-indexer - This is a tool to take our search.ndjson file and index this data into a postgres database
  • /search/worker - A cloudflare worker that can query the postgres database to provide search functionality.

@Yantrio Yantrio requested review from damianstasik, Evi1Pumpkin and a user August 28, 2024 13:22
cam72cam
cam72cam reviewed Aug 28, 2024
View reviewed changes
search/pg-indexer/main.go Outdated Show resolved Hide resolved
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Aug 28, 2024 •
edited
Loading

Deploying registry-ui with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4886b92
Status: ✅  Deploy successful!
Preview URL: https://8f017a1d.registry-ui-dxi.pages.dev
Branch Preview URL: https://search-index.registry-ui-dxi.pages.dev

View logs

ghost
ghost suggested changes Aug 29, 2024
View reviewed changes
search/pg-indexer/main.go Show resolved Hide resolved
search/pg-indexer/schema.sql Show resolved Hide resolved
search/pg-indexer/types.go Show resolved Hide resolved
search/worker/.editorconfig Outdated Show resolved Hide resolved
search/worker/tsconfig.json Outdated Show resolved Hide resolved
@Yantrio Yantrio merged commit 195c7f4 into main Aug 29, 2024
4 checks passed
@Yantrio Yantrio deleted the search-index branch August 29, 2024 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cam72cam cam72cam cam72cam left review comments

@damianstasik damianstasik Awaiting requested review from damianstasik

@Evi1Pumpkin Evi1Pumpkin Awaiting requested review from Evi1Pumpkin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Yantrio @cam72cam