You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently query parsing is being done both by various duplicate QueryFilter classes spread throughout the resources files and also by a central QueryParser class in utils.py that is inherited from to create query parsers for EBS Snapshots, IAM Policies, and Cloud Directories. I believe this issue stems from there never having been a distinction made between query filters and query parameters. Most enum spec functions support query parameters to perform server-side filtering of results. A few of these functions (such as describe_instances for ec2) also offer a Filters parameter which takes a list of Name, Values dictionaries and allows you to perform server-side filtering for a much larger list of parameters. Because these two different types of queries were never distinguished, a lot of unnecessary and confusing translations between dictionaries and lists of Name, Values dictionaries are being done and many potential server-side queries are not supported.
It seems to me that all query parsing should be centralized into the QueryParser class, thus greatly simplifying a lot of this code and providing a number of new server-side query possibilities such as CreationTime.
Extra information or context
No response
The text was updated successfully, but these errors were encountered:
Describe the feature
Currently query parsing is being done both by various duplicate QueryFilter classes spread throughout the resources files and also by a central QueryParser class in utils.py that is inherited from to create query parsers for EBS Snapshots, IAM Policies, and Cloud Directories. I believe this issue stems from there never having been a distinction made between query filters and query parameters. Most enum spec functions support query parameters to perform server-side filtering of results. A few of these functions (such as describe_instances for ec2) also offer a
Filters
parameter which takes a list ofName
,Values
dictionaries and allows you to perform server-side filtering for a much larger list of parameters. Because these two different types of queries were never distinguished, a lot of unnecessary and confusing translations between dictionaries and lists ofName
,Values
dictionaries are being done and many potential server-side queries are not supported.It seems to me that all query parsing should be centralized into the QueryParser class, thus greatly simplifying a lot of this code and providing a number of new server-side query possibilities such as CreationTime.
Extra information or context
No response
The text was updated successfully, but these errors were encountered: