Skip to content
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

compile inline query doesn't add node #7292

Closed
wants to merge 10 commits into from
Closed

Conversation

ChenyuLInx
Copy link
Contributor

@ChenyuLInx ChenyuLInx commented Apr 6, 2023

Related to #6358
When compile a inline node, we add one additional node to the manifest. That node should be removed when compile is done.

@cla-bot cla-bot bot added the cla:yes label Apr 6, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Apr 6, 2023

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

@ChenyuLInx ChenyuLInx force-pushed the cl/compile_query_work branch from 69fb6bf to 47b915d Compare April 6, 2023 22:14
# it should be removed before the task is complete
_inline_node_id = None

# TODO remove when stu's PR about skip adapter cache is merged
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As stated, this should be removed once Stu's work is done.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tracked in #6526

Copy link
Contributor

@jtcohen6 jtcohen6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2023-04-10 at 14 01 49

you simply love to see it

# it should be removed before the task is complete
_inline_node_id = None

# TODO remove when stu's PR about skip adapter cache is merged
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tracked in #6526


return runner.safe_run(manifest)


def parse_to_manifest(config):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we implement #6547, we would be able to remove this method as well, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep

@ChenyuLInx ChenyuLInx force-pushed the cl/compile_query_work branch from 47b915d to 762a3f6 Compare April 11, 2023 18:11
@ChenyuLInx ChenyuLInx changed the title dbt compile work with server, remove lib.py associated with it compile inline query doesn't add node Apr 11, 2023
@ChenyuLInx ChenyuLInx closed this Apr 11, 2023
@ChenyuLInx
Copy link
Contributor Author

Closing due to git history looks odd, will reopen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants