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

docs(blog): needle haystack post #8824

Merged
merged 6 commits into from
Apr 12, 2024

Conversation

IndexSeek
Copy link
Member

Description of changes

Here is the post we chatted about in Zulip.

I tried to set up a scenario to make it a bit more interesting to a reader, but I am happy to make any changes or go more in-depth with explanations.

@lostmygithubaccount lostmygithubaccount added docs Documentation related issues or PRs docs-preview Add this label to trigger a docs preview labels Mar 29, 2024
@ibis-docs-bot ibis-docs-bot bot removed the docs-preview Add this label to trigger a docs preview label Mar 29, 2024
@ibis-docs-bot
Copy link

ibis-docs-bot bot commented Mar 29, 2024

Copy link
Member

@cpcloud cpcloud left a comment

Choose a reason for hiding this comment

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

Sorry for the delay in review!

Just a few style nits and then this is good to go!

docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Show resolved Hide resolved
docs/posts/varchar-in-a-haystack/index.qmd Outdated Show resolved Hide resolved
@cpcloud
Copy link
Member

cpcloud commented Apr 12, 2024

To expedite this I will commit the review suggestions.

@cpcloud cpcloud added this to the 9.0 milestone Apr 12, 2024
@cpcloud cpcloud added the blog Posts for the Ibis blog. label Apr 12, 2024
@cpcloud cpcloud enabled auto-merge (squash) April 12, 2024 11:10
@cpcloud cpcloud merged commit ca97867 into ibis-project:main Apr 12, 2024
16 checks passed
@IndexSeek
Copy link
Member Author

Sorry for the delay in review!

Just a few style nits and then this is good to go!

No worries at all! Thank you for the expedition, it would have otherwise been a little while for me probably.

Looking forward to checking it out and sharing from the site.

@IndexSeek IndexSeek deleted the blog/varchar-stack branch October 30, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blog Posts for the Ibis blog. docs Documentation related issues or PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants