Feat: Add environments command to list environments #3770
Conversation
| @cli_analytics | ||
| def environments(obj: Context, expiry_ds: bool) -> None: | ||
| """Prints the list of SQLMesh environments with its expiration datetime.""" | ||
| environments = {e.name: e.expiration_ts for e in obj.state_sync.get_environments()} |
There was a problem hiding this comment.
Just a heads up that environment objects are pretty large and loading and parsing them all at once can be a pretty long and memory-intensive operation for big projects with a lot of environments.
A better approach could be having a specialized method in the the state_sync.py that only fetches limited fields like names.
There was a problem hiding this comment.
sure, I'll do this. Thanks :)
| environment_names = list(environments.keys()) | ||
| if expiry_ds: | ||
| max_width = len(max(environment_names, key=len)) | ||
| print( |
There was a problem hiding this comment.
I think we should be using the console instance here as well.
| ) | ||
| @line_magic | ||
| @pass_sqlmesh_context | ||
| def environments(self, context: Context, line: str) -> None: |
There was a problem hiding this comment.
This looks like the exact copy-paste of the code above. It might be worse considering having a method as part of Context that prints this.
There was a problem hiding this comment.
I followed the pattern used by table_name method in magics.py.
If you not mind could you please elaborate on whats expected here?
|
Thanks a lot for your contribution! Left some comments |
Fixes #3732
I came across this feature request and found it interesting. I made some assumptions and created this PR. Please review and share your feedback. thanks :)