Skip to content
New issue

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

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

Already on GitHub? # to your account

Blogpost about Hybrid Search #100

Merged
merged 2 commits into from
Jun 21, 2024
Merged

Conversation

lizbiella
Copy link
Contributor

Added the blogpost about Hybrid Search

Added the blogpost about Hybrid Search
@alessandrobenedetti
Copy link
Contributor

alessandrobenedetti commented Jun 17, 2024

@lizbiella works with me and we tried together to open this first PR.
We are not sure this is the right process to link external blog posts, once we are sure, we'll contribute all Sease's blog posts around Solr (and they are a decent bunch)

Copy link
Contributor

@janhoy janhoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I spun the site up and found your post did not look like the templates provided. See other comments.
Skjermbilde 2024-06-20 kl  12 39 56
Skjermbilde 2024-06-20 kl  12 40 02

I think it will help to add a summary: metadata and a level-1 title.

@gerlowskija
Copy link
Contributor

We are not sure this is the right process to link external blog posts

Just commenting to affirm - at least IMO, what you guys have done all follows what we landed on in SOLR-16833 👍

Copy link
Contributor

@gerlowskija gerlowskija left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Marking as 'Request changes' for now, until Jan's formatting comments have been addressed. But then I'm happy to 'Approve'.

Thanks for adding the first external post to the blog - very exciting!

Applied changes after review
@lizbiella
Copy link
Contributor Author

Thanks @janhoy and @gerlowskija for the comments! I addressed all your concerns and updated the pull request! Let me know if there is something else to change

@alessandrobenedetti alessandrobenedetti merged commit 1d04240 into apache:main Jun 21, 2024
@alessandrobenedetti
Copy link
Contributor

Merged! We have plenty (probably >50 Solr blog post in our archive).
Our plan is to contribute one every few day, so the Solr blog will be populated soon :D

Thanks @lizbiella for the contribution and @janhoy and @gerlowskija for the reviews, the next pull requests will be easier!

@janhoy
Copy link
Contributor

janhoy commented Jun 21, 2024

Nice @alessandrobenedetti, it is now live on the staging site https://solr.staged.apache.org/blog.html:
Skjermbilde 2024-06-21 kl  10 46 56

In order to publish it, you have to merge main branch into production branch. You can do it through a PR or from command line, but make sure it is a normal merge with a merge commit, no squashing or cherry picking.

@chatman
Copy link

chatman commented Jun 21, 2024 via email

@alessandrobenedetti
Copy link
Contributor

This blog shows how to run a hybrid search (keyword-based search +
vectors) in Apache Solr with code examples and explanations. Thanks Alessandro. Can we change "blog shows" to "blog post shows" here?

On Fri, 21 Jun, 2024, 2:21 pm Jan Høydahl, @.> wrote: Nice @alessandrobenedetti https://github.com/alessandrobenedetti, it is now live on the staging site https://solr.staged.apache.org/blog.html: Skjermbilde.2024-06-21.kl.10.46.56.png (view on web) https://github.com/apache/solr-site/assets/409128/a548879d-5558-49c3-8e9c-49a9917d1015 In order to publish it, you have to merge main branch into production branch. You can do it through a PR or from command line, but make sure it is a normal merge with a merge commit, no squashing or cherry picking. — Reply to this email directly, view it on GitHub <#100 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDCR5BFJWUL7VH6E5U55J3ZIPSP5AVCNFSM6AAAAABJOB5UMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBSGMYDSMJXGE . You are receiving this because you are subscribed to this thread.Message ID: @.>

thanks @chatman , just merged this fix into the production branch!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants