Open amanangira opened 4 months ago
@amanangira Please help confirm that my understanding is correct. To support the snowflake in sql instrumentation, we can add an new API likeaddDetector()
, if you don't mind, could you help submit a PR for it? We can review and add unit testing for the changes.
Hello @wangzlei , I don't mind. I sure can try and respond to the review changes as they are reviewed. Before moving forward I wanted to confirm if there was no existing API to do the same.
Are there any instructions to keep in mind when working on a new PR? Thanks!
Hello, below are the details of what I am trying to do
During debugging the
QueryContext
context statement I found the below call stack.Now the problem as per my understanding is that the
[newDBAttribute](https://github.com/aws/aws-xray-sdk-go/blob/e9627da6f47902f96f26c577cbe6f9244987a4fb/xray/sql_context.go#L370)
call under sql_context.go only seems to be supportingpostgres
,mysql
, andoracle
driver types because of which the meta query being fired in case of snowflake isSELECT version(), current_user, current_database()
and the Snowflakeversion()
equivalent is[current_version()](https://docs.snowflake.com/en/sql-reference/functions/current_version)
which results in some error logs.Version I am using go 1.21.0 aws-xray-sdk-go v1.8.3 snowflakedb/gosnowflake v1.8.0