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

Remove mut from raw_query #339

Merged
merged 1 commit into from
Jul 5, 2024
Merged

Remove mut from raw_query #339

merged 1 commit into from
Jul 5, 2024

Conversation

rijkvp
Copy link
Contributor

@rijkvp rijkvp commented Jun 13, 2024

The &mut reference was not required here, this solved a use case for us where we wanted to query while the statement was immutably borrowed.

@Mytherin Mytherin merged commit 1c5e7cd into duckdb:main Jul 5, 2024
4 checks passed
@Mytherin
Copy link
Contributor

Mytherin commented Jul 5, 2024

Thanks!

robert3005 referenced this pull request in spiraldb/vortex Jul 11, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [duckdb](https://github.com/wangfenjin/duckdb-rs) |
workspace.dependencies | major | `0.10.2` -> `1.0.0` |

---

### Release Notes

<details>
<summary>wangfenjin/duckdb-rs (duckdb)</summary>

###
[`v1.0.0`](https://github.com/duckdb/duckdb-rs/releases/tag/v1.0.0)

[Compare
Source](https://github.com/wangfenjin/duckdb-rs/compare/v0.10.2...v1.0.0)

This release upgrades the shipped DuckDB to v1.0.0, in addition to a
number of bug fixes.

**Note**: The `httpfs` module that ships the `httpfs` extension is
removed in this release. The `httpfs` extension can instead be used
through DuckDB's native extension installation/loading mechanism.

#### What's Changed

- feat: support "large" arrow data types by
[@&#8203;Mause](https://github.com/Mause) in
[https://github.com/duckdb/duckdb-rs/pull/307](https://github.com/duckdb/duckdb-rs/pull/307)
- Fixes the issue ignoring nulls when copy from arrow array to flat
vector by [@&#8203;y-f-u](https://github.com/y-f-u) in
[https://github.com/duckdb/duckdb-rs/pull/316](https://github.com/duckdb/duckdb-rs/pull/316)
- change SQLite to DuckDB by
[@&#8203;era127](https://github.com/era127) in
[https://github.com/duckdb/duckdb-rs/pull/308](https://github.com/duckdb/duckdb-rs/pull/308)
- Organise workspace crates and consolidate Cargo.toml manifest
properties/dependencies by
[@&#8203;Swoorup](https://github.com/Swoorup) in
[https://github.com/duckdb/duckdb-rs/pull/285](https://github.com/duckdb/duckdb-rs/pull/285)
- Update README.md by [@&#8203;yoonghm](https://github.com/yoonghm) in
[https://github.com/duckdb/duckdb-rs/pull/242](https://github.com/duckdb/duckdb-rs/pull/242)
- Add support for DuckDB arrays when using Arrow's FixedSizeList by
[@&#8203;Jeadie](https://github.com/Jeadie) in
[https://github.com/duckdb/duckdb-rs/pull/323](https://github.com/duckdb/duckdb-rs/pull/323)
- Add support for BinaryArray in arrow-vtab by
[@&#8203;phillipleblanc](https://github.com/phillipleblanc) in
[https://github.com/duckdb/duckdb-rs/pull/324](https://github.com/duckdb/duckdb-rs/pull/324)
- use openssl from runner by
[@&#8203;Maxxen](https://github.com/Maxxen) in
[https://github.com/duckdb/duckdb-rs/pull/327](https://github.com/duckdb/duckdb-rs/pull/327)
- Add ability to pass vendored feature to openssl in libduckdb-sys by
[@&#8203;muralisoundararajan](https://github.com/muralisoundararajan)
in
[https://github.com/duckdb/duckdb-rs/pull/321](https://github.com/duckdb/duckdb-rs/pull/321)
- Implement appender for date/time types by
[@&#8203;rijkvp](https://github.com/rijkvp) in
[https://github.com/duckdb/duckdb-rs/pull/313](https://github.com/duckdb/duckdb-rs/pull/313)
- Support decimal128 without casting to double by
[@&#8203;Maxxen](https://github.com/Maxxen) in
[https://github.com/duckdb/duckdb-rs/pull/328](https://github.com/duckdb/duckdb-rs/pull/328)
- feat: experimental support for further nested types by
[@&#8203;Mause](https://github.com/Mause) in
[https://github.com/duckdb/duckdb-rs/pull/318](https://github.com/duckdb/duckdb-rs/pull/318)
- upgrade arrow-rs to 52 by [@&#8203;Mause](https://github.com/Mause)
in
[https://github.com/duckdb/duckdb-rs/pull/332](https://github.com/duckdb/duckdb-rs/pull/332)
- Expose the underlying schema of executed statement by
[@&#8203;abhiaagarwal](https://github.com/abhiaagarwal) in
[https://github.com/duckdb/duckdb-rs/pull/333](https://github.com/duckdb/duckdb-rs/pull/333)
- Remove mut from raw_query by
[@&#8203;rijkvp](https://github.com/rijkvp) in
[https://github.com/duckdb/duckdb-rs/pull/339](https://github.com/duckdb/duckdb-rs/pull/339)
- Support Arrow type `LargeUtf8`. by
[@&#8203;Jeadie](https://github.com/Jeadie) in
[https://github.com/duckdb/duckdb-rs/pull/341](https://github.com/duckdb/duckdb-rs/pull/341)
- Update duckdb-rs to DuckDB v1.0.0 by
[@&#8203;Mytherin](https://github.com/Mytherin) in
[https://github.com/duckdb/duckdb-rs/pull/336](https://github.com/duckdb/duckdb-rs/pull/336)
- fix: don't write files outside of `OUT_DIR` by
[@&#8203;frectonz](https://github.com/frectonz) in
[https://github.com/duckdb/duckdb-rs/pull/347](https://github.com/duckdb/duckdb-rs/pull/347)
- Remove built-in HTTPFS extension and fix Windows CI by
[@&#8203;Mytherin](https://github.com/Mytherin) in
[https://github.com/duckdb/duckdb-rs/pull/353](https://github.com/duckdb/duckdb-rs/pull/353)
- Upgrade duckdb-loadable-macros to 0.1.2 by
[@&#8203;Mytherin](https://github.com/Mytherin) in
[https://github.com/duckdb/duckdb-rs/pull/354](https://github.com/duckdb/duckdb-rs/pull/354)

#### New Contributors

- [@&#8203;y-f-u](https://github.com/y-f-u) made their first
contribution in
[https://github.com/duckdb/duckdb-rs/pull/316](https://github.com/duckdb/duckdb-rs/pull/316)
- [@&#8203;era127](https://github.com/era127) made their first
contribution in
[https://github.com/duckdb/duckdb-rs/pull/308](https://github.com/duckdb/duckdb-rs/pull/308)
-
[@&#8203;muralisoundararajan](https://github.com/muralisoundararajan)
made their first contribution in
[https://github.com/duckdb/duckdb-rs/pull/321](https://github.com/duckdb/duckdb-rs/pull/321)
- [@&#8203;rijkvp](https://github.com/rijkvp) made their first
contribution in
[https://github.com/duckdb/duckdb-rs/pull/313](https://github.com/duckdb/duckdb-rs/pull/313)
- [@&#8203;abhiaagarwal](https://github.com/abhiaagarwal) made their
first contribution in
[https://github.com/duckdb/duckdb-rs/pull/333](https://github.com/duckdb/duckdb-rs/pull/333)
- [@&#8203;frectonz](https://github.com/frectonz) made their first
contribution in
[https://github.com/duckdb/duckdb-rs/pull/347](https://github.com/duckdb/duckdb-rs/pull/347)

**Full Changelog**:
duckdb/duckdb-rs@v0.10.2...v1.0.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/spiraldb/vortex).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MjUuMSIsInVwZGF0ZWRJblZlciI6IjM3LjQyNS4xIiwidGFyZ2V0QnJhbmNoIjoiZGV2ZWxvcCIsImxhYmVscyI6W119-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Robert Kruszewski <github@robertk.io>
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