Setting interceptor provider does not include default Spanner interceprtors

[nielm]

The line setting the interceptor provider at CaptureGfeMetric.java:L67 does not include the default spanner interceptors (error handling and logging)

        .setInterceptorProvider(() -> Collections.singletonList(interceptor))

should be

        setInterceptorProvider(SpannerInterceptorProvider.createDefault().with(interceptor))

[nielm]

Actually it looks like since Oct 7th, we log metrics for GFE latency already... googleapis/java-spanner#1473

[nielm]

@mayurkale22

[eaball35]

@jsimonweb Any updates? This issue is out of SLO.

[nielm]

Fixed by #6177 - thanks @KiranmayiB

[KiranmayiB]

Could you please explain? I am out of the loop. @nielm

[nielm]

I raised an issue about one of the Spanner samples. In the meantime you fixed it with PR #6177 :)
So this issue can be closed!

[KiranmayiB]

got it. thank you :)

…

On Wed, Jan 19, 2022 at 5:00 PM Niel Markwick ***@***.***> wrote: I raised an issue about one of the Spanner samples. In the meantime you fixed it with PR #6177 <#6177> :) So this issue can be closed! — Reply to this email directly, view it on GitHub <#6441 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/APWES2DLYWLWI6YHBIHREBDUW2ODTANCNFSM5JEW4CWA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Thanks Kiranmayi