feat(ingest): looker - improve defaults for usage extraction now that… #5893

shirshanka · 2022-09-09T17:57:45Z

… connector is stable

Checklist

The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
Links to related issues (if applicable)
Tests for the changes have been added/updated (if applicable)
Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

… connector is stable

metadata-ingestion/src/datahub/ingestion/source/looker/looker_source.py

github-actions · 2022-09-09T18:28:43Z

      8 files       8 suites 55m 26s ⏱️
  685 tests   682 ✔️ 3 💤 0 ❌
1 370 runs 1 364 ✔️ 6 💤 0 ❌

Results for commit 31366ee.

hsheth2

Lgtm once surya’s suggestion is done

github-actions · 2022-09-09T18:29:37Z

571 tests +60 571 ✔️ +61 14m 19s ⏱️ +14s
141 suites +21     0 💤 ±  0
141 files +21     0 ❌ -   1

Results for commit 31366ee. ± Comparison against base commit 5a0154f.

…source.py Co-authored-by: Ravindra Lanka <[email protected]>

feat(ingest): looker - improve defaults for usage extraction now that…

31366ee

… connector is stable

shirshanka requested a review from hsheth2 September 9, 2022 17:57

rslanka reviewed Sep 9, 2022

View reviewed changes

metadata-ingestion/src/datahub/ingestion/source/looker/looker_source.py Outdated Show resolved Hide resolved

hsheth2 approved these changes Sep 9, 2022

View reviewed changes

Update metadata-ingestion/src/datahub/ingestion/source/looker/looker_…

ba0c4dd

…source.py Co-authored-by: Ravindra Lanka <[email protected]>

shirshanka merged commit 5074035 into datahub-project:master Sep 9, 2022