Copilot commented on code in PR #1475:
URL:
https://github.com/apache/datafusion-python/pull/1475#discussion_r3034080664
##########
python/datafusion/context.py:
##########
@@ -1141,6 +1142,67 @@ def session_id(self) -> str:
"""Return an id that uniquely identifies this
:py:class:`SessionContext`."""
return self.ctx.session_id()
+ def session_start_time(self) -> str:
+ """Return the session start time as an RFC 3339 formatted string."""
+ return self.ctx.session_start_time()
+
+ def enable_ident_normalization(self) -> bool:
+ """Return whether identifier normalization (lowercasing) is enabled."""
+ return self.ctx.enable_ident_normalization()
+
Review Comment:
The PR description says it closes #1459, but the issue’s checklist also
includes `add_optimizer_rule` and `add_analyzer_rule`. Those methods still
don’t appear to be exposed in either the Rust bindings or the Python wrapper
(no references found in the repo). Either add the remaining methods or adjust
the PR description / linked issue closure to reflect the reduced scope.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
