-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed
Description
Goal
Strengthen validation of CLI arguments and improve error reporting across the insert-tools CLI (mainly clickhouse_insert.cli and runner).
Tasks
- Validate required arguments in one place (e.g.,
host,target_table,select_sql) - Add checks for missing or conflicting arguments
- Handle errors such as:
- Missing
select_sqlvalue - Non-existent file if reading SQL from file
- Invalid types (e.g.,
portis not an integer)
- Missing
- Improve error messages to be more user-friendly
- Add
validate()method insideInsertConfigto encapsulate checks
Why it matters
Current CLI behavior lacks robust validation and gives unclear errors when arguments are missing or misused. Improving this will make the tool easier to use and reduce confusion.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed