docs: Improved documentation for read_sql_query #3135
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
akhil0404-tech
changed the title
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
Thanks, however the change must be made in the code directly not in the md file. |
|
Hi Jaidisido,
Thanks for your feedback! I apologize for the confusion.
1.
Regarding the code changes, I’ll update the code directly instead of the
markdown file (md).
2.
Regarding the Athena reference: You're absolutely right! I mistakenly
referred to Athena, while the code relates more to MySQL. I will update the
documentation to accurately reflect MySQL instead.
I'll go ahead and make the necessary adjustments. Thanks again for pointing
that out!
Best regards,
Akhil Chatla
…On Wed, 23 Apr 2025 at 05:44, jaidisido ***@***.***> wrote:
Thanks, however the change must be made in the code directly not in the md
file.
Also you are referring to Athena in your description but your code relates
more to MySQL?
—
Reply to this email directly, view it on GitHub
<#3135 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BKXCRGH4HYEGGYG6OIRQBH3225OGVAVCNFSM6AAAAAB3UYHVA6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQMRTG4YTGOBUHA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
*jaidisido* left a comment (aws/aws-sdk-pandas#3135)
<#3135 (comment)>
Thanks, however the change must be made in the code directly not in the md
file.
Also you are referring to Athena in your description but your code relates
more to MySQL?
—
Reply to this email directly, view it on GitHub
<#3135 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BKXCRGH4HYEGGYG6OIRQBH3225OGVAVCNFSM6AAAAAB3UYHVA6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQMRTG4YTGOBUHA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Feature
read_sql_queryDetails
read_sql_queryin theawswrangler.athenamodule.