Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

aws-log: [Minor improvement] add newline between each statement in CloudWatch Log Saved Query #22422

Open
2 tasks
khaing211 opened this issue Oct 7, 2022 · 4 comments
Labels
@aws-cdk/aws-cloudwatch Related to Amazon CloudWatch effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2

Comments

@khaing211
Copy link

Describe the feature

Could someone add newline between each statement in CloudWatch Log Saved Query?

https://github.com/aws/aws-cdk/blob/v2.45.0/packages/@aws-cdk/aws-logs/lib/query-definition.ts#L105

This format makes this easy for someone to use because the query fits in the AWS Console code editor screen.

Use Case

I'm always frustrated when I have to scroll horizontally, in the code editor on CloudWatch Log Insight Query, to find my placeholder in the Saved Query.

Proposed Solution

No response

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CDK version used

2.x

Environment details (OS name and version, etc.)

NA

@khaing211 khaing211 added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Oct 7, 2022
@github-actions github-actions bot added the @aws-cdk/aws-cloudwatch Related to Amazon CloudWatch label Oct 7, 2022
@indrora
Copy link
Contributor

indrora commented Nov 16, 2022

I'm not sure I'm following what the use case here is... Can you explain some more?

@indrora indrora added p2 effort/small Small work item – less than a day of effort and removed needs-triage This issue or PR still needs to be triaged. labels Nov 16, 2022
@khaing211
Copy link
Author

khaing211 commented Nov 16, 2022 via email

@jerry-shao
Copy link
Member

Hi @madeline-k ,

I have raised a PR to improve the readability of Logs Insights Query Definition for this issue: #23094
Any chance you could have a look?

Thanks!

mergify bot pushed a commit that referenced this issue Nov 26, 2022
…#23094)

fix(logs): Adding line break for CloudWatch Logs Insights Query Definition (#22422)

Verified in AWS Console with integration test, we can see query in multiple lines instead of a single line.

<img width="1159" alt="image" src="https://user-images.githubusercontent.com/7830699/204059301-afd0adb7-b24e-4588-a0aa-58fd0cdbbfd8.png">

----

### All Submissions:

* [✅] Have you followed the guidelines in our [Contributing guide?](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md)

### Adding new Unconventional Dependencies:

* [❌] This PR adds new unconventional dependencies following the process described [here](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md/#adding-new-unconventional-dependencies)

### New Features

* [✅] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/main/INTEGRATION_TESTS.md)?
	* [✅] Did you use `yarn integ` to deploy the infrastructure and generate the snapshot (i.e. `yarn integ` without `--dry-run`)?

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@madeline-k madeline-k removed their assignment Oct 30, 2023
@jerry-shao
Copy link
Member

Hi @pahud

This issue has been resolve by this PR: #23094

I think we can resolve this issue now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-cloudwatch Related to Amazon CloudWatch effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2
Projects
None yet
Development

No branches or pull requests

4 participants