chore: change SqlExecutor node name to SQLExecutor

[TrachukT]

Important

Renames SqlExecutor to SQLExecutor and updates AWSRedshift default port to 5439.

• Renaming:
  • Rename SqlExecutor to SQLExecutor in sql_executor.py, __init__.py, use_sql.py, and test_sql_executor.py.
• Behavior:
  • Update AWSRedshift default port from 5432 to 5439 in connections.py.
• Misc:
  • Add query attribute to SQLExecutor class in sql_executor.py to handle cases where input data does not provide a query.

^{This description was created by for b40d89a. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 3535cf4 in 10 seconds

More details

• Looked at 83 lines of code in 4 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. dynamiq/nodes/tools/__init__.py:10

• Draft comment:
  Ensure that all instances of SqlExecutor are updated to SQLExecutor throughout the codebase to prevent runtime errors. This includes imports and any references in documentation or comments.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR changes the class name from SqlExecutor to SQLExecutor in multiple files. This is a straightforward change, but it is important to ensure that all instances of the old class name are updated to prevent runtime errors.

Workflow ID: wflow_GTDLbVq2dP2T5FIB

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[github-actions]

Coverage Report •

File	Stmts	Miss	Cover	Missing
dynamiq/connections
connections.py	381	46	87%	14–18, 58, 66, 77, 100, 464–465, 467–468, 472–473, 475–476, 489–490, 492, 518, 520, 532, 534–536, 574–575, 593–594, 610–611, 644–645, 652, 752, 805, 850–851, 862, 956–957, 990–991, 1027–1028
dynamiq/nodes/tools
__init__.py	12	0	100%
sql_executor.py	34	4	88%	51, 59–61
TOTAL	10770	3317	69%

Tests	Skipped	Failures	Errors	Time
347	0 💤	0 ❌	0 🔥	48.891s ⏱️

[ellipsis-dev]

👍 Looks good to me! Incremental review on 55ed568 in 16 seconds

More details

• Looked at 46 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. dynamiq/connections/connections.py:1002

• Draft comment:
  The default port for AWS Redshift has been correctly updated to 5439, which is the standard port for Redshift.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The change in the default port for AWSRedshift from 5432 to 5439 is correct as 5439 is the default port for Redshift.

2. dynamiq/nodes/tools/sql_executor.py:14

• Draft comment:
  Changing the default value of query to an empty string allows for instantiation without providing a query, but ensure that this does not lead to unintended behavior elsewhere in the code. The execute method does handle empty queries by raising a ValueError.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The change in the default value for the query field in SQLInputSchema from a required field to an empty string is a significant change. This could lead to potential issues if the query is not provided, as the code now allows for an empty query to be passed without raising an error at the schema level. However, the code does handle empty queries in the execute method by raising a ValueError.

Workflow ID: wflow_KmajbdTyBbLKhE1Q

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

👍 Looks good to me! Incremental review on b40d89a in 37 seconds

More details

• Looked at 22 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. dynamiq/nodes/tools/sql_executor.py:14

• Draft comment:
  For compatibility with Python versions below 3.10, consider using Optional[str] instead of str | None. This applies to line 36 as well.

    query: Optional[str] = Field(None, description="Parameter to provide a query that needs to be executed.")

• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:
  The comment is technically correct - using Optional[str] would make the code compatible with older Python versions. However, this seems like a codebase-wide decision rather than a file-specific issue. The use of | syntax suggests the codebase has already standardized on Python 3.10+. Making this change in isolation wouldn't make sense.
  The comment points out a real compatibility issue. If the project needs to support Python <3.10, this would be a blocker.
  However, the consistent use of | syntax throughout the file and lack of Optional import suggests Python 3.10+ is an intentional requirement. Making this change in just one place would be inconsistent.
  Delete the comment since it appears to conflict with the codebase's apparent Python version requirements and would create inconsistency if implemented only here.

Workflow ID: wflow_eHTR3gajJ0smWxya

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.