Skip to content

Install full text search as an option#35

Merged
rbev merged 8 commits into
mainfrom
rhys/sql-fulltext
Jul 22, 2026
Merged

Install full text search as an option#35
rbev merged 8 commits into
mainfrom
rhys/sql-fulltext

Conversation

@rbev

@rbev rbev commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

This PR adds full text search support to the SQL server installed by this action.

Enable it using the enable-full-text-search parameter

@rbev
rbev force-pushed the rhys/sql-fulltext branch from d817ff4 to dacde12 Compare July 21, 2026 05:48
@rbev
rbev marked this pull request as ready for review July 21, 2026 07:20

@johnsimons johnsimons left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So how much more time does enabling FTS takes?

Comment thread .github/workflows/ci.yml Outdated
@rbev

rbev commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author

So how much more time does enabling FTS takes?

Linux: ~55s
Windows: nil

@johnsimons

Copy link
Copy Markdown
Member

Linux: ~55s
Windows: nil

The only time Windows is faster 🤣

@rbev

rbev commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author

Linux: ~55s
Windows: nil

The only time Windows is faster 🤣

Sample of one so grain of salt....

Platform FTS Time
Windows 3.50
Windows 3.43
Linux 0:25
Linux 1:21

@rbev
rbev enabled auto-merge July 22, 2026 03:24
@rbev
rbev merged commit d91a5a7 into main Jul 22, 2026
13 checks passed
@rbev
rbev deleted the rhys/sql-fulltext branch July 22, 2026 03:29
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.

2 participants