Skip to content

Fix error reporting in seqcli query#461

Merged
nblumhardt merged 4 commits into
datalust:devfrom
nblumhardt:query-syntax-errors
Jun 3, 2026
Merged

Fix error reporting in seqcli query#461
nblumhardt merged 4 commits into
datalust:devfrom
nblumhardt:query-syntax-errors

Conversation

@nblumhardt
Copy link
Copy Markdown
Member

@nblumhardt nblumhardt commented Jun 3, 2026

> seqcli query -q "select count(*) from stream where a != b group by @Level"
The query could not be executed.

 - Syntax error (line 1, column 37): unsupported operator `!=`.                                                                                                                      
 - SQL queries only support `=`, `<>`, `and`, `or`, and `not` in logical expressions.                                                                                                
                                                                                                                                                                                     
Perhaps you meant:                                                                                                                                                                   
                                                                                                                                                                                     
select count(*) from stream where a <> b group by @Level

@nblumhardt nblumhardt marked this pull request as draft June 3, 2026 04:18
@nblumhardt
Copy link
Copy Markdown
Member Author

Just spotted that I still need to remove the helper and update the MCP tool.

Comment thread src/SeqCli/Csv/CsvWriter.cs Outdated
theme.Reset(output);
}

WriteCell(output, theme, heading, isHeadingRow: true);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: Since WriteCell already accepts some context to tell whether it's a heading or not, it seems reasonable to fold the , separator in there too.

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.

Done, munched down to much less code

@nblumhardt nblumhardt marked this pull request as ready for review June 3, 2026 04:33
@nblumhardt nblumhardt merged commit df3c094 into datalust:dev Jun 3, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants