Skip to content

Fix various trino query things - #355

Open
Bohreromir wants to merge 5 commits into
mainfrom
fix-trino-query-length-display
Open

Bohreromir wants to merge 5 commits into
mainfrom
fix-trino-query-length-display

Conversation

@Bohreromir

Copy link
Copy Markdown
Member

All in seperate commits, I hope its readable that way

Fixes:

  • querytimer display to fall back to ms instead of showing 0s
  • Wrongly flagged exactly MAX_CLIENT_ROWS
  • the query button opening the dropdown
  • Layout shifts when running a query

Also reports the case of being over the row limit to the user

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 50.98% 2503 / 4909
🔵 Statements 57.05% 3720 / 6520
🔵 Functions 65.71% 1031 / 1569
🔵 Branches 42.97% 1303 / 3032
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/lib/components/trino/StatementResult.svelte 0% 0% 0% 0% 21-284
src/lib/server/trino/queries.ts 13.23% 17.5% 15.78% 13.6% 48, 52, 68, 83-336
src/lib/server/trino/result-collector.ts 69.38% 55.17% 100% 73.91% 24, 30-36, 39, 44, 64-67, 90
src/lib/types/query.ts 100% 100% 100% 100%
src/routes/(app)/trino/+page.svelte 0% 0% 0% 0% 404-940
src/routes/(app)/trino/query-runner.svelte.ts 0% 0% 0% 0% 38-331
Generated in workflow #1040 for commit 82fe2f1 by the Vitest Coverage Report Action

@Bohreromir Bohreromir changed the title Fix trino query length display Fix various trino query things Oct 6, 2026
@Bohreromir
Bohreromir marked this pull request as ready for review October 6, 2026 16:58
res.writeHead(204).end();
return;
}
if (req.method === 'GET' && req.url?.startsWith('/v1/statement/executing/q-huge/')) {

@Bohreromir Bohreromir Oct 6, 2026 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not the nicest way, but I have the feeling we'll refactor the test mocking at some point anyway

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Waiting for Review

Development

Successfully merging this pull request may close these issues.

1 participant