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

Ability to add more comments with the Commenter Feature #192

Open
debuggerpk opened this issue Nov 2, 2023 · 1 comment
Open

Ability to add more comments with the Commenter Feature #192

debuggerpk opened this issue Nov 2, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@debuggerpk
Copy link

GCP CloudSQL Commenter add a couple of keys e.g. action, route etc which are visible in CloudSQL Query insights. In those keys, action key is very interesting. It would be wonderful if we can add more comments to the query.

I have been playing the library a little bit but I can't seem to wrap to head around this line.

c.propagator.Inject(ctx, &cc)

My debugging investigation shows that traceparent becomes available in context after this line. I believe this happens because we are instantiating it with propagation.NewCompositeTextMapPropagator()

Is there a way we can set more keys for commentCarrier when setting the context of the query?

PS: thankyou for the wonderful work.

@debuggerpk debuggerpk added the enhancement New feature or request label Nov 2, 2023
@XSAM
Copy link
Owner

XSAM commented Nov 6, 2023

Hi @debuggerpk, thanks for raising this issue.

I am not sure about the scenario that custom comments could solve. Could you please describe more about the new keys that CloudSQL Commenter introduced?

  • What kind of value could be stored under these keys?
  • What UI or system did you expect to see those keys?
  • What reasons could adding these keys to comments in SQL help you solve the problem? Would adding attributes into the span before calling the SQL operation do the same for you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants