[pull] trunk from spiceai:trunk#41
Merged
Merged
Conversation
…(spice run) (#5464) * Upgrade Go from v1.23.4 to v1.24.2 * Enable captured outputs by default when spiced is started by the CLI (spice run) - Also refactor rtcontext and remove runtime.go * Remove debug trace * Fix linting issue * Use rtcontext
* add parameters to QueryBuilder and Query * add new param_utils module for converting json to paramvalues * add parameters to logical plan; add caching tests * check value of queries in tests * add hook in do_put for prepared statements * wip * wip * allow null placeholder fields in schema verify * fix clippy lint for CI * add comment about null placeholder skip * use snafu * add comment about do_put_query * move record_to_param_values to arrow_tools crate * add unit tests for record_to_param_values * make decode_param_values simpler * simplify testing of param values * track flight requests * fix clippy pedantic lints * make flight server stateless * remove schema calculation * fix regular do_put after making Peekable stream * use expect instead of unwrap for clippy lints * fix clippy lints * fix more clippy lints * parse u64 first * treat ? as placeholder in verify schema * use postcard to serialize parameterized queries * fix tests after parsing uint first * fix lint
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
3 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.1)
Can you help keep this open source service alive? 💖 Please sponsor : )