Skip to content

SQLSource parameter binding doesn't support passing entire SQL queries, only scalar values - needs clearer documentation #139

@loukratz-bv

Description

@loukratz-bv

Problem

When using SQLSource, it's not clear from the documentation that parameters are for binding scalar VALUES, not for injecting entire SQL query strings.

Reproduction

Given a Python tool that returns a SQL query string:

def build_query(keywords: str) -> str:
    return f"SELECT * FROM table WHERE name LIKE '%{keywords}%'"

And a SQLSource step attempting to use this query:

- type: InvokeTool
  id: build_query
  tool: my_tools.build_query
  input_bindings:
    keywords: keywords
  output_bindings:
    build_query_result: sql_query

- type: SQLSource
  id: execute_query
  connection: "awsathena+rest://..."
  query: ":sql_query"
  inputs:
    - sql_query
  outputs:
    - result_id

Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## ExpectTYP## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Expected ## Exant## Expected ##g t## Expected ## Expected ## Expected ## Expected ## res## Expected with parameterized VALUES only (e.g., WHERE id = :product_id).

Suggested Documentation Improvement

Add cleAdd cleAdd cleAdd cleAdd cleAdd cleAdd cleAdd cle. ✅ Correct:Add cleAdd cleAddpaAdd cleAdd cleAuesAdd cleAdd cleAdd cleAdd clngAdd cleAdd cleAdSQAdd cleAdd cleAdd clers

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions