Skip to content

Commit

Permalink
Clarify solution of lab_4.3.sql
Browse files Browse the repository at this point in the history
It is not clear that step6's query is expected to fail. As we updated the question to suggest it might not work, I am updating the solutions to explicitly say that it doesn't work and why.
  • Loading branch information
pmusa authored Sep 9, 2024
1 parent a0d427f commit 46e335a
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion developer/04_inserting_data/lab_4.3.sql
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,11 @@ SETTINGS
format_csv_delimiter='~',
schema_inference_hints='approved_amount UInt32, recommended_amount UInt32';

/*
* No, the command above does not work. It fails becuase the data contains `n/a`
* in the `approved_amount` that cannot be parsed to UInt32.
*/

--Step 7:
CREATE TABLE operating_budget (
fiscal_year LowCardinality(String),
Expand Down Expand Up @@ -105,4 +110,4 @@ WHERE fiscal_year = '2022';
SELECT sum(actual_amount)
FROM operating_budget
WHERE fiscal_year = '2022'
AND program_code = '031';
AND program_code = '031';

0 comments on commit 46e335a

Please sign in to comment.