-
Notifications
You must be signed in to change notification settings - Fork 867
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
[discover] pass connection title correctly and set session id #7997
Conversation
Updated so the session ID gets correctly passed along to speed up subsequent queries. Also set the connection title to the datasource meta field to be used as originally interfaced. Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
❌ Empty Changelog SectionThe Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7997 +/- ##
==========================================
+ Coverage 61.09% 61.13% +0.04%
==========================================
Files 3691 3691
Lines 87310 87342 +32
Branches 13433 13435 +2
==========================================
+ Hits 53340 53398 +58
+ Misses 30722 30690 -32
- Partials 3248 3254 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Updated so the session ID gets correctly passed along to speed up subsequent queries. Also set the connection title to the datasource meta field to be used as originally interfaced. Signed-off-by: Kawika Avilla <kavilla414@gmail.com> (cherry picked from commit de027c2) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…#8002) Updated so the session ID gets correctly passed along to speed up subsequent queries. Also set the connection title to the datasource meta field to be used as originally interfaced. (cherry picked from commit de027c2) Signed-off-by: Kawika Avilla <kavilla414@gmail.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Updated so the session ID gets correctly passed along to speed up subsequent queries. Also set the connection title to the datasource meta field to be used as originally interfaced. Signed-off-by: Kawika Avilla <kavilla414@gmail.com> (cherry picked from commit de027c2) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…#8007) Updated so the session ID gets correctly passed along to speed up subsequent queries. Also set the connection title to the datasource meta field to be used as originally interfaced. (cherry picked from commit de027c2) Signed-off-by: Kawika Avilla <kavilla414@gmail.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
Updated so the session ID gets correctly passed along to speed up subsequent queries. Also set the connection title to the datasource meta field to be used as originally interfaced.
Issues Resolved
n/a
Screenshot
Testing the changes
Changelog
Check List
yarn test:jest
yarn test:jest_integration