Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

rows_result: implement into_inner behind cpp_rust_unstable cfg #1133

Merged

Conversation

muzarski
Copy link
Contributor

@muzarski muzarski commented Nov 29, 2024

This is convenience introduced only for cpp-rust-driver purposes.

We will use DeserializedMetadataAndRawRows directly to implement lazy deserialization in cpp-rust-driver

Pre-review checklist

  • I have split my patch into logically separate commits.
  • All commit messages clearly explain what they change and why.
  • [ ] I added relevant tests for new features and bug fixes.
  • All commits compile, pass static checks and pass test.
  • PR description sums up the changes and reasons why they should be introduced.
  • [ ] I have provided docstrings for the public items that I want to introduce.
  • [ ] I have adjusted the documentation in ./docs/source/.
  • [ ] I added appropriate Fixes: annotations to PR description.

@muzarski muzarski self-assigned this Nov 29, 2024
@muzarski muzarski added the cpp-rust-driver-p0 Functionality required by cpp-rust-driver label Nov 29, 2024
Copy link

github-actions bot commented Nov 29, 2024

cargo semver-checks found no API-breaking changes in this PR! 🎉🥳
Checked commit: ead6ef0

scylla/Cargo.toml Outdated Show resolved Hide resolved
@muzarski muzarski force-pushed the cpp-rust-decouple-query-rows-result branch from 618657c to 33a5455 Compare November 29, 2024 14:16
@muzarski muzarski requested a review from Lorak-mmk November 29, 2024 14:17
@muzarski
Copy link
Contributor Author

muzarski commented Nov 29, 2024

v2: Changed cpp-rust feature to cpp_rust_unstable cfg option. Added clippy check with the cfg enabled to CI.

This is convenience introduced only for cpp-rust-driver purposes.

We will use DeserializedMetadataAndRawRows directly to implement
lazy deserialization in cpp-rust-driver
@muzarski muzarski force-pushed the cpp-rust-decouple-query-rows-result branch from 33a5455 to ead6ef0 Compare December 2, 2024 14:15
@muzarski muzarski changed the title rows_result: implement into_inner behind cpp-rust feature rows_result: implement into_inner behind cpp_rust_unstable cfg Dec 2, 2024
@muzarski
Copy link
Contributor Author

muzarski commented Dec 2, 2024

v2.1: rebased on main (CI fixes) and adjusted PR title

@wprzytula wprzytula requested a review from Lorak-mmk December 2, 2024 14:44
@Lorak-mmk Lorak-mmk merged commit 62f96b3 into scylladb:main Dec 2, 2024
11 checks passed
wprzytula pushed a commit to wprzytula/scylla-rust-driver that referenced this pull request Dec 11, 2024
…y-rows-result

rows_result: implement into_inner behind `cpp_rust_unstable` cfg

(cherry picked from commit 62f96b3)
@wprzytula wprzytula mentioned this pull request Dec 11, 2024
@muzarski muzarski deleted the cpp-rust-decouple-query-rows-result branch December 19, 2024 16:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
cpp-rust-driver-p0 Functionality required by cpp-rust-driver
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants