[pull] master from cube-js:master - #621
Merged
Merged
Conversation
* feat(cubejs-cli): show CLI version when run without a command Running `cubejs` with no arguments now prints the CLI version above the help output, so users can see which version they have installed at a glance. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01McCxp9LtvvZ5cMhStkHfS1 * Revert "feat(cubejs-cli): show CLI version when run without a command" This reverts commit 1a369cc. * feat(cube-cli): show CLI version when run without a command Running bare `cube` now prints "Cube CLI <version>" followed by the full help text, instead of clap's missing-subcommand error. The subcommand is now optional at the parser level and handled early in main, before the update check and telemetry. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01McCxp9LtvvZ5cMhStkHfS1 --------- Co-authored-by: Claude <noreply@anthropic.com>
…atus (#11321) * test(api-gateway): reproduce data-source-unaware pre-agg job queue status (#11313) getPreAggJobQueueStatus calls orchestrator.getPreAggregationQueueStates() without job.dataSource, so QueryOrchestrator falls back to its 'default' parameter and never finds queue entries for jobs on named data sources. Adds a failing test pinning this down at the gateway level. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WTFSSej9SP4JTbWBxZKgBq * fix(api-gateway): pass job data source when checking pre-agg queue status getPreAggJobQueueStatus called orchestrator.getPreAggregationQueueStates() without job.dataSource, so QueryOrchestrator fell back to its 'default' parameter and only ever inspected the default data source's queue. Jobs running on named (non-default) data sources were never matched, so their status fell through to the table-existence check and reported missing_partition for the entire duration of a healthy build instead of processing. Fixes #11313 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WTFSSej9SP4JTbWBxZKgBq * fix(server-core): forward dataSource in OrchestratorApi.getPreAggregationQueueStates The api-gateway now passes job.dataSource when checking pre-agg job queue status, but OrchestratorApi.getPreAggregationQueueStates dropped the argument at its boundary, so QueryOrchestrator still fell back to the 'default' data source at runtime. Fixes #11313 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WTFSSej9SP4JTbWBxZKgBq * chore: re-trigger CI Birdbox pre-aggregations job failed on a postgres fixture-load race (psql exit 2 before the container finished init), unrelated to the diff. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WTFSSej9SP4JTbWBxZKgBq --------- Co-authored-by: Claude <noreply@anthropic.com>
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.
1 participant
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 : )