emecii commented on PR #1716: URL: https://github.com/apache/datafusion-python/pull/1716#issuecomment-5669605851
You're right: #970 explicitly requests `aws.*` SQL options and already reports successful Python registration. Removed the closure claim. Reproducing its SQL still gives `Could not find config namespace "aws"`, even with a registered store. The CLI adds AWS option processing and object-store creation around SQL; Python's SQL binding delegates to the core context. Pushed 0d0a29e with user-facing workflow text, simplified environment credential setup, and a network-free SQL registry test (26 object-store tests pass). Both docs examples also read three rows through loopback Moto with dummy credentials; this does not validate real AWS authentication or SDK credential-chain parity. Should this PR remain a docs improvement, or should it instead implement CLI-style SQL AWS options to address #970? -- 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]
