Antonio-RiveroMartnez commented on code in PR #22024:
URL: https://github.com/apache/superset/pull/22024#discussion_r1015949367
##########
tests/unit_tests/db_engine_specs/test_bigquery.py:
##########
@@ -244,3 +244,74 @@ def test_mask_encrypted_extra_when_empty() -> None:
from superset.db_engine_specs.bigquery import BigQueryEngineSpec
assert BigQueryEngineSpec.mask_encrypted_extra(None) is None
+
+
+def test_parse_error_message() -> None:
+ """
+ Test that we parse a received message and just extract the useful
information.
+
+ Example errors:
+ 400 Syntax error: Expected "(" or keyword UNNEST but got "@" at [4:80]
+
+ (job ID: 60c732c3-4b4e-4da6-9988-18ba20d389ee)
+
+ -----Query Job SQL Follows-----
+
+ | . | . | . | . | . | . | . |
. | . | . | . | . | . | . | . |
+ 1:SELECT count(*) AS `count_1`
+ 2:FROM (SELECT
`bigquery-public-data.census_bureau_acs.blockgroup_2010_5yr`.`geo_id` AS
`bigquery_public_data_census_bureau_acs_blockgroup_2010_5yr_geo_id`
+ 3:FROM `bigquery-public-data.census_bureau_acs.blockgroup_2010_5yr`
+ 4:WHERE
`bigquery-public-data.census_bureau_acs.blockgroup_2010_5yr`.`geo_id` IN
@`geo_id_1`) AS `anon_1`
+ | . | . | . | . | . | . | . |
. | . | . | . | . | . | . | . |
+
+
+ bigquery error: 400 Table \"case_detail_all_suites\" must be qualified
with a dataset (e.g. dataset.table).
Review Comment:
Updated 👍
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org
