-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Java V2 Added an example for tix V1818828107 #7486
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
Conversation
Please build the PR to your CDD and put the link in the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand why this file and all the rest show the top part as deleted, when it looks like all you did was add some javadoc to the method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The problem is that this example doesn't use the describeLogStreams method, which is the API method/page it appears on in the Code Examples library.
If the the intent of the example is to display logged content, this should be moved to a different place, because here it is misleading.
Also, this class contains several unused imports.
This pull request adds a new code example for tix V1818828107, .
Cloud Desktop Build is available.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.