-
Notifications
You must be signed in to change notification settings - Fork 7
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Description
The Query class (and the related Field class) from the query_builder module has some functionality to display information about a query at hand and which fields,enums etc. one might add to it. A lot of this functionality is related to the .get_info method.
Currently this method prints the information to stdout when it is called. This has two main disadvantages.
- The method name is misleading as it doesn't "get" anything but instead "prints" something
- It's difficult to build applications on top of the query_builder class that make use of the
.get_infoinformation in their own way (i.e. don't do naive printing). Such an application could be a dashboard for data exploration.
I would suggest to refactor .get_info and related methods to return a data type, for instance a dict or a named tumple containing the information, such that it can be build upon. Addionally one could extend the Query/Field class to additionally retain its info printing funcitonality based on the new .get_info but under a different name. For instance .print_info.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request