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

Remove UDF declarations from queries #159

Closed
jklukas opened this issue May 28, 2019 · 2 comments
Closed

Remove UDF declarations from queries #159

jklukas opened this issue May 28, 2019 · 2 comments
Assignees

Comments

@jklukas
Copy link
Contributor

jklukas commented May 28, 2019

Follow-up to #141

  • Extend the code in tests/parse_udfs.py to also read through the files under sql/ and identify usage of udfs there
  • Write a python script to use the above udf dependency resolution logic, writing out generated sql files to a new directory (perhaps we call this target/sql to match Maven's conventions); each file would contain the body of all temporary UDFs needed in the query, and then the query text itself
  • Remove UDF declarations from the source files under sql/ and update the logic for our CircleCI deploy job to make sure that the published container has a sql/ directory that contains the generated queries with UDF definitions
@jklukas
Copy link
Contributor Author

jklukas commented May 28, 2019

This would supersede #148

@scholtzan
Copy link
Collaborator

I'll work on this.

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

No branches or pull requests

2 participants