feature: Graphql

[obecny]

Which problem is this PR solving?

• Adding graphql instrumentation
• Fixes GraphQL Apollo Server #14

Short description of the changes

• It instruments the graphql library
• Added example of using Apollo Server and Express Graphql - 2 most popular lib.

[codecov]

Codecov Report

Merging #226 into master will decrease coverage by 0.19%.
The diff coverage is 93.88%.

@@            Coverage Diff             @@
##           master     #226      +/-   ##
==========================================
- Coverage   95.26%   95.06%   -0.20%     
==========================================
  Files         100      110      +10     
  Lines        5028     5879     +851     
  Branches      526      605      +79     
==========================================
+ Hits         4790     5589     +799     
- Misses        238      290      +52     

Impacted Files	Coverage Δ
...opentelemetry-instrumentation-graphql/.eslintrc.js	0.00% <0.00%> (ø)
...entelemetry-instrumentation-graphql/test/schema.ts	55.84% <55.84%> (ø)
...entelemetry-instrumentation-graphql/src/graphql.ts	92.36% <92.36%> (ø)
...opentelemetry-instrumentation-graphql/src/utils.ts	96.17% <96.17%> (ø)
.../opentelemetry-instrumentation-graphql/src/enum.ts	100.00% <100.00%> (ø)
...entelemetry-instrumentation-graphql/src/symbols.ts	100.00% <100.00%> (ø)
...opentelemetry-instrumentation-graphql/src/types.ts	100.00% <100.00%> (ø)
...entelemetry-instrumentation-graphql/src/version.ts	100.00% <100.00%> (ø)
...metry-instrumentation-graphql/test/graphql.test.ts	100.00% <100.00%> (ø)
...entelemetry-instrumentation-graphql/test/helper.ts	100.00% <100.00%> (ø)
... and 10 more

[dyladan]

I already reviewed this when the PR was open in the main repo but I'll put the same comment here.

I'm not familiar with the graphql module's API, but it seems like a generally straightforward implementation of the tracing and the use of the new instrumentation module looks good to me. Approving.

[tvvignesh]

@obecny Thanks a lot for this. Just looked up on NPM and I am not able to find the package @opentelemetry/instrumentation-graphql - Is it yet to be pushed?

Also, does this support Apollo Federation? Thanks

[obecny]

@obecny Thanks a lot for this. Just looked up on NPM and I am not able to find the package @opentelemetry/instrumentation-graphql - Is it yet to be pushed?

it hasn't been released yet

Also, does this support Apollo Federation? Thanks

Not sure, the plugin instruments the graphql core library, so anything that is using that should be instrumented the same. but if certain library does something more beyond the graphql lib then not

[PacoDu]

@obecny Thanks a lot for this. Just looked up on NPM and I am not able to find the package @opentelemetry/instrumentation-graphql - Is it yet to be pushed?

it hasn't been released yet

Could we know when this will be released ? Looking forward to try it out !

[hongbo-miao]

Just saw it published six days ago.
Tested and it works great, thanks @obecny !

[carlitux]

@obecny any suggestion to debug?, I have all set up done but nothing is traced.

[obecny]

@obecny any suggestion to debug?, I have all set up done but nothing is traced.

Please create an issue and fill out all needed information including code samples etc. otherwise it is impossible to tell anything