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

rename BQLCompiler to Environmemnt in compiler.py #536

Open
riastradh-probcomp opened this issue Feb 28, 2017 · 0 comments
Open

rename BQLCompiler to Environmemnt in compiler.py #536

riastradh-probcomp opened this issue Feb 28, 2017 · 0 comments
Labels

Comments

@riastradh-probcomp
Copy link
Contributor

We should acknowledge what the object is, and use this to factor out some common functionality. Instead of a monolithic compile_bql method, we should replace it by individual methods like population_id, implicit_rowid, &c., and deduplicate code generation for calls to bql_*.

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

No branches or pull requests

1 participant