-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(ingestion/lookml): liquid template resolution and view-to-view cll #10542
Merged
hsheth2
merged 65 commits into
datahub-project:master
from
sid-acryl:master+ing-510-lookml-cll
Jul 8, 2024
Merged
Changes from 55 commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
32a6ab0
wip
sid-acryl b308f91
introduce datahub sqlparser for sql parsing
sid-acryl d09b296
remove import pdb
sid-acryl 3010df0
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 4dedd87
syntax1 and syntax2 support of lookml sql view
sid-acryl 7922cc7
update message
sid-acryl 86026c6
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 9f9d510
add if else for syntax
sid-acryl 93add15
Merge branch 'master+ing-510-lookml-cll' of github.com:sid-acryl/data…
sid-acryl 9dbd32b
liquid variable resolution
sid-acryl 4f44fd9
liquid variable in config
sid-acryl fe96655
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 522d4a5
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl fcd9957
sqlglot_lib
sid-acryl 808ded1
Merge branch 'master+ing-510-lookml-cll' of github.com:sid-acryl/data…
sid-acryl 542de95
test fixes
sid-acryl ab61277
view to view lineage
sid-acryl 734fbbe
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 7e75198
drop hive. from urn id
sid-acryl 3d2b13b
Merge branch 'master+ing-510-lookml-cll' of github.com:sid-acryl/data…
sid-acryl 2e6bffd
name.SQL_TABLE_NAME handling
sid-acryl 5f9a582
added debug log
sid-acryl dc231a6
add stack trace
sid-acryl 4cc0048
fix for select * from something.SQL_TABLE_NAME
sid-acryl 54aff14
fix test cases
sid-acryl 6c79b37
test case
sid-acryl b77eb3f
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl d58600f
rephrased description
sid-acryl 1415b10
Merge branch 'master+ing-510-lookml-cll' of github.com:sid-acryl/data…
sid-acryl 853ca3d
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 3cc0487
code refactoring to resolve circular dependency
sid-acryl 27c3861
Merge branch 'master+ing-510-lookml-cll' of github.com:sid-acryl/data…
sid-acryl 85c324b
Added case base logic
sid-acryl 16fc4c9
refactored code
sid-acryl 6cc7c60
refactored code
sid-acryl 2b297c5
WIP
sid-acryl fd943a4
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 93a8413
Merge branch 'master+ing-510-lookml-cll' of github.com:sid-acryl/data…
sid-acryl 2f61c16
WIP
sid-acryl a368a4b
refactor code
sid-acryl cfbba07
remove unused py file
sid-acryl 045f769
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 84fa6c9
updated test case for custom tag
sid-acryl 31e784b
Merge branch 'master+ing-510-lookml-cll' of github.com:sid-acryl/data…
sid-acryl 7094405
update test case
sid-acryl ae02aae
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl b693251
handle special liquid variable
sid-acryl 853fc25
Add double quotes to cover one customer use case
sid-acryl 17f6204
add lru cache
sid-acryl 6ae99c4
test case for sql_table_name having - in name
sid-acryl d72f475
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 0854446
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 12859b0
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 659dd3a
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 5ad8200
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 2e8c14f
refactor code
sid-acryl b9f8b08
fix existing golden files
sid-acryl cf0f45b
1. Resolve merge conflict
sid-acryl 33985b5
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 519c173
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 0b926d0
resolve merge conflict
sid-acryl d5025d4
Merge branch 'master+ing-510-lookml-cll' of github.com:sid-acryl/data…
sid-acryl c2d2f6b
address review comments
sid-acryl e7008d2
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl 8629f42
Merge branch 'master' into master+ing-510-lookml-cll
sid-acryl File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.