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

Statement add new API for execute to clean Vala/GObject API #1286

Closed
wants to merge 2 commits into from

Conversation

esodan
Copy link
Contributor

@esodan esodan commented Nov 12, 2023

Currently Statement.execute() use void * to pass a ArrowArrayStream, which in turn is not defined in GLib Arrow, also this force manual memory management when using Vala, so this new methods, clean up the way to better use on Vala and GLib/GObject

This is depends on #1152

Vala VAPI file is generated for current GADBC 1.0 API version.

It is disable by default, but recommended to be enable when
a development package is distributed.

This commit force a 1.0 API versioning, replacing the old one
where 0 were used for GIR API version (gir_version), so now
shows '1.0' instead of '0.0'.

Also examples are provided for C and Vala.
* New Statement.execute_query() for queries without results or ignoring them
* New Statement.execute_reader() for queries returning results and converting
them to GArrowRecordBatchReader

Above methods allows to ignore extrange API current in Statement.execute()
when using on Vala
@esodan esodan requested a review from kou as a code owner November 12, 2023 05:28
Copy link

⚠️ Please follow the Conventional Commits format in CONTRIBUTING.md for PR titles.

@kou
Copy link
Member

kou commented Jan 22, 2024

Solved by #1459.

@kou kou closed this Jan 22, 2024
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