[pull] trunk from spiceai:trunk#715
Merged
Merged
Conversation
* feat: add support for custom authentication header in GraphQL client * fix: address PR review - fix auth logic, add tests, revert unrelated changes - Fix auth selection: Basic auth now works even when auth_header is set but token is missing - Add 9 unit tests: auth selection logic + request_with_auth behavior for all variants - Add 2 integration tests: custom auth header verification, json_pointer + query combinations - Revert unrelated changes to cayenne, datafusion/mod.rs, and data_components/lib.rs * fix: add warning when auth_header is set but auth_token is missing * Address PR review: use prefixed param names in messages, improve test error handling * fix: Apply cargo fmt to tracing::warn macro call * Address PR review: generic warning messages, warn on auth_header without credentials, refactor test server helper * fix: Correct test_graphql_json_pointer_combinations assertion to match SELECT query
* Fix ADBC Catalog * fix * Fix * Fix * Add more basic integration tests for distributed accelerations (#9740) * more basic tests for distributed accelerations * Refactor distributed acceleration tests to use async file operations and improve wait logic * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Improve wait_for_row_count robustness and fix doc comment * Fix lint errors and update inline snapshots for distributed acceleration tests - Replace #[allow] with #[expect] (clippy::allow-attributes) - Add backticks around TEST_DATA_CSV in doc comment (clippy::doc-markdown) - Replace match with if let for single-pattern destructuring (clippy::single-match) - Add #[expect(clippy::cast_possible_truncation)] for i64-to-usize cast - Update inline snapshots to include bucket partition filters in plans * formatting * fix: use r#"..."# raw strings for insta snapshots containing double quotes Entire-Checkpoint: 7f5359852068 * fix snapshots --------- Co-authored-by: Luke Kim <80174+lukekim@users.noreply.github.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Refactor ADBC functions to be crate-private and update tests for driver options * Refactor import statements in adbc.rs --------- Co-authored-by: Jack Eadie <jack@spice.ai> Co-authored-by: Luke Kim <80174+lukekim@users.noreply.github.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
* Add --endpoint flag to spice run with scheme-based routing Add a new --endpoint flag that auto-routes to the appropriate endpoint based on URL scheme: - http:// or https:// -> HTTP endpoint (--http) - grpc:// or grpc+tls:// -> Flight endpoint (--flight) Returns an error if no recognized scheme is provided, or if --endpoint conflicts with an already-specified --http-endpoint or --flight-endpoint. Fixes #9901 * Fix rustfmt formatting in resolve_endpoint
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )