Another Looker/LookML bug we've spotted:
we are a...
# ingestionp
plain-farmer-27314
03/07/2022, 9:29 PMAnother Looker/LookML bug we've spotted:
we are also finding examples of Charts that don't have lineage to their explores
Interestingly our staging environment has the correct lineage (and is running a much older version of the looker/lookml ingestors)
Can see them in the properties field, but no lineage is actually being established to these explores
g
green-football-43791
03/07/2022, 10:53 PMHey @plain-farmer-27314 would you be able to share the query that populated the chart / more information on the explore? My guess is something w/ the setup of this chart is hard for us to parse.
p
plain-farmer-27314
03/08/2022, 2:01 PMI can - but based on the image it looks like the explore's fields were correctly parsed, they just weren't sent as lineage events
plain-farmer-27314
03/08/2022, 2:02 PMThe explore itself also has correct lineage to the view and further, so its just the chart <-> explore connection that's missing
plain-farmer-27314
03/08/2022, 8:03 PMBump - what's the best way I can provide info on the above chart?
g
green-football-43791
03/09/2022, 12:22 AMHey @plain-farmer-27314 - is that screenshot you shared w/ the
upstream_fieldsgreen-football-43791
03/09/2022, 12:23 AMbut you’re saying that chart has 0 upstream entities?
green-football-43791
03/09/2022, 1:04 AMalso cc @dazzling-judge-80093 - any thoughts on what could be the issue here?
d
dazzling-judge-80093
03/09/2022, 7:55 AMI think with the query we can make sure it is not related to the sql parser or actually you can switch to the old query parser with :
sql_parser: datahub.utilities.sql_parser.MetadataSQLSQLParserp
plain-farmer-27314
03/09/2022, 2:32 PMI will say I have a staging environment that's running an older version of the looker ingest, and the lineage was parsed correctly
plain-farmer-27314
03/09/2022, 2:33 PM@green-football-43791 correct, the screenshot contains the upsteam_fields from the chart entity, and yet it has 0 upstream entities
plain-farmer-27314
03/09/2022, 2:33 PMGonna run with older version of the SQLParser
plain-farmer-27314
03/09/2022, 2:35 PM@dazzling-judge-80093 is that config in the looker or lookml ingest? i don't see it here: https://datahubproject.io/docs/metadata-ingestion/source_docs/looker
d
dazzling-judge-80093
03/09/2022, 3:02 PMAhh it’s only for lookml -> https://datahubproject.io/docs/metadata-ingestion/source_docs/lookml
2 Views