ralph icon indicating copy to clipboard operation
ralph copied to clipboard

Accept set when dealing with list of IDs

Open lebaudantoine opened this issue 2 years ago • 0 comments

Feature Request

Is your feature request related to a problem or unsupported use case? Please describe. Using the BaseDatabase class, I encountered an issue where I had to convert a set of IDs into a list to match the accepted data type. This conversion seemed unnecessary since sets are more suitable for dealing with unique IDs. Converting a set to a list adds unnecessary overhead, and if possible, we should avoid it to improve efficiency.

This enhancement might be needed elsewhere in the codebase.

Describe the solution you'd like Enhance the query_statements_by_ids here method's API to allow the relevant data structures for the ids parameter.

lebaudantoine avatar Sep 20 '23 14:09 lebaudantoine