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.

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

Post: quarkus.search.io series - rolling over #1898

Merged

Conversation

marko-bekhta
Copy link
Contributor

@yrodiere here it is 😃 🙈

Copy link

github-actions bot commented Feb 26, 2024

🙈 The PR is closed and the preview is expired.

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Looks very good overall, I added a few suggestions.

_posts/2024-02-25-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-02-25-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-02-25-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-02-25-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-02-25-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
link:https://github.com/quarkusio/search.quarkus.io/blob/d956b6a1341d8693fa1d6b7881f3840f48bdaacd/src/main/java/io/quarkus/search/app/indexing/Rollover.java#L44-L331[rollover implementation]
and link:https://github.com/quarkusio/search.quarkus.io/blob/d956b6a1341d8693fa1d6b7881f3840f48bdaacd/src/main/java/io/quarkus/search/app/indexing/IndexingService.java#L226-L244[rollover usage].

Now, since we would want to keep our application providing results to any search operations and add/update documents to the indexes
Copy link
Member

Choose a reason for hiding this comment

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

I think you need to provide some context about how the documentation is indexed and the fact that you are mass indexing the documentation every time and not updating documents (which would be another approach and in this case you probably wouldn't need this approach).

_posts/2024-02-25-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
@marko-bekhta marko-bekhta force-pushed the post/search-quarkus-io-rollover branch from 1a30f0c to ca2d700 Compare April 24, 2024 20:04
@marko-bekhta
Copy link
Contributor Author

Thanks Guillaume! I've tried to address your suggestions.

Copy link
Member

@yrodiere yrodiere left a comment

Choose a reason for hiding this comment

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

Looks great! I'm curious: what did you use for the drawings?

I added a few suggestions below, feel free to pick what you agree with.

On important thing though: the feature request for rollover already exists, it's https://hibernate.atlassian.net/browse/HSEARCH-3499 . I added suggestions accordingly.

_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
_posts/2024-04-30-search-indexing-rollover.adoc Outdated Show resolved Hide resolved
@marko-bekhta marko-bekhta force-pushed the post/search-quarkus-io-rollover branch from ca2d700 to 7a23b79 Compare April 25, 2024 10:02
@marko-bekhta marko-bekhta marked this pull request as ready for review April 25, 2024 10:03
@marko-bekhta
Copy link
Contributor Author

Yoann, thanks for the suggestions! I've applied them all (at least I think I did 😄)

as for the drawing tool, I was trying out this one https://excalidraw.com/

@marko-bekhta
Copy link
Contributor Author

I'll let you decide on the date when you'd like to publish this one 😃

@yrodiere
Copy link
Member

Thanks. Well then, seems we're ready to merge? We'll just need to agree on a date so that @marko-bekhta can update the post's metadata before we merge. @gsmet is there some process or is anyone free to post whenever they like?

@gsmet
Copy link
Member

gsmet commented Apr 25, 2024

You can post whenever you like. Thursday is OK. I don't recommend Monday or Friday. I was told around posting around 3PM was the best (if you plan on tweeting about it).

I can either retweet or tweet (if you write something for me) with the QuarkusIO account if you'd like. But please ping me on Zulip when the blog post is published so that I don't have to monitor things :).

@yrodiere
Copy link
Member

@marko-bekhta today 3PM then?

@gsmet
Copy link
Member

gsmet commented Apr 25, 2024

Obviously you can push the blog post earlier. 3PM is more about when we should tweet.

@marko-bekhta marko-bekhta force-pushed the post/search-quarkus-io-rollover branch from 7a23b79 to a7f909c Compare April 25, 2024 10:56
@marko-bekhta
Copy link
Contributor Author

@marko-bekhta today 3PM then?

Updated the file name and the date.
Or should I use the date with time? Not sure how that works 😃

@yrodiere yrodiere merged commit cc0c6a1 into quarkusio:develop Apr 25, 2024
1 check passed
@yrodiere
Copy link
Member

Merged, thanks. @marko-bekhta I'll let you tweet about this and/or provide Guillaume with a tweet for the @quarkusio account on X.

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

Successfully merging this pull request may close these issues.

3 participants