Skip to content

common: fix linting failures#2146

Merged
stephenplusplus merged 1 commit intogoogleapis:masterfrom
stephenplusplus:spp--ci-fixes
Mar 28, 2017
Merged

common: fix linting failures#2146
stephenplusplus merged 1 commit intogoogleapis:masterfrom
stephenplusplus:spp--ci-fixes

Conversation

@stephenplusplus
Copy link
Contributor

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 28, 2017
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6e11db1 on stephenplusplus:spp--ci-fixes into ** on GoogleCloudPlatform:master**.

@stephenplusplus stephenplusplus merged commit 2cc3836 into googleapis:master Mar 28, 2017
sofisl pushed a commit that referenced this pull request Feb 25, 2026
…2146)

* test: add debug flag for suppressing extra console messages in ssb

* remove unneeded third argument
sofisl pushed a commit that referenced this pull request Feb 26, 2026
…l.createSessions (#2145)

This change adds tracing for Database.batchCreateSessions
as well as SessionPool.createSessions which was raised
as a big need.
This change is a premise to finishing up tracing Transaction.

While here, also folded in the async/await fix to avoid day+ long
code review lag and then 3+ hours just to run tests per PR:
OpenTelemetry cannot work correctly for async/await if there isn't
a set AsyncHooksManager, but we should not burden our customers with
this type of specialist knowledge, their code should just work and
this change performs such a check. Later on we shall file a feature
request with the OpenTelemetry-JS API group to give us a hook to detect
if we've got a live asyncHooksManager instead of this mandatory
comparison to ROOT_CONTEXT each time.

Fixes #2146
Updates #2079
Spun out of PR #2122
Supersedes PR #2147
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants