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

Bypass string_agg on jruby until jdbc-sqlite3 supports it #404

Conversation

martinemde
Copy link
Member

@martinemde martinemde commented Feb 25, 2025

Description:

Aggregate dependencies the old fashioned way on jruby until jdbc-sqlite3 supports string_agg.

I will abide by the code of conduct.

@martinemde martinemde force-pushed the martinemde/bypass-string-agg-for-compact-index-on-jruby branch 7 times, most recently from 1bd6506 to 5d38d90 Compare February 25, 2025 17:11
@martinemde martinemde changed the title Add an ugly bypass of string_agg on jruby Bypass string_agg on jruby until jdbc-sqlite3 supports it Feb 25, 2025
@martinemde martinemde force-pushed the martinemde/bypass-string-agg-for-compact-index-on-jruby branch 2 times, most recently from 5bcc22f to 50e4d64 Compare February 25, 2025 17:18
@segiddins segiddins force-pushed the segiddins/add-compact-index-support-for-private-sources branch from 10aeb06 to 770c564 Compare February 25, 2025 19:24
@martinemde martinemde force-pushed the martinemde/bypass-string-agg-for-compact-index-on-jruby branch from 779c598 to 675c398 Compare February 25, 2025 22:55
@martinemde martinemde requested a review from segiddins February 25, 2025 23:10
@martinemde martinemde force-pushed the martinemde/bypass-string-agg-for-compact-index-on-jruby branch from 675c398 to 4930ff6 Compare February 25, 2025 23:11
@martinemde
Copy link
Member Author

Looks like PR is passing string_agg now. Closing

@martinemde martinemde closed this Feb 27, 2025
@martinemde martinemde deleted the martinemde/bypass-string-agg-for-compact-index-on-jruby branch February 27, 2025 06:50
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.

1 participant