-
Notifications
You must be signed in to change notification settings - Fork 7.4k
[Data] Allow parameterized queries in read_sql #54098
Copy link
Copy link
Closed
Labels
P2Important issue, but not time-criticalImportant issue, but not time-criticaldataRay Data-related issuesRay Data-related issuesdata:datasourcesdata:sqlenhancementRequest for new feature and/or capabilityRequest for new feature and/or capabilitygood-first-issueGreat starter issue for someone just starting to contribute to RayGreat starter issue for someone just starting to contribute to Ray
Description
Description
Allow parameterized queries when using read_sql to construct a Dataset.
Use case
Currently, the query string passed to read_sql must be fully formed and not contain any parameter bindings. This limits its applicability both in dev (need to use raw python string formatting for parameters, opportunities for mistakes) and in prod (SQL injection). It should be possible to use a parameterized query string and pass a dict/list of parameters alongside the query.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P2Important issue, but not time-criticalImportant issue, but not time-criticaldataRay Data-related issuesRay Data-related issuesdata:datasourcesdata:sqlenhancementRequest for new feature and/or capabilityRequest for new feature and/or capabilitygood-first-issueGreat starter issue for someone just starting to contribute to RayGreat starter issue for someone just starting to contribute to Ray