-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Support for Firebolt data source #5606
Support for Firebolt data source #5606
Conversation
…id/redash into firebolt-integration
…id/redash into firebolt-integration
Hi, |
Hi Redash team! I'm leading Firebolt's integrations team and Rajesh has been working with me to add this connector. Would super appreciate if someone could take a look at this PR and merge it if all looks good. @arikfr @kravets-levko @susodapop |
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.
Thanks for this contribution!
Thanks for pinging us @rajeshSigmoid and @kevinmarr . Busy days with the V10 release and redash-migrate. It's good to get a reminder 😃 |
Thank you!! Appreciate it.
…On Thu, Oct 14, 2021 at 10:57 AM Jesse ***@***.***> wrote:
Thanks for pinging us @rajeshSigmoid <https://github.com/rajeshSigmoid>
and @kevinmarr <https://github.com/kevinmarr> . Busy days with the V10
release and redash-migrate. It's good to get a reminder 😃
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5606 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB2RVONDWGU7KVSWADV2UBDUG4K2LANCNFSM5FCKFETA>
.
|
@susodapop When will the next release happen that includes this PR? Should we be telling our customers to build directly from source code while we wait for the next release? |
Will aim to include this in the V10.1 release shortly. |
Great, thank you! Now, for the annoying follow-up question: when will the V10.1 release go out? |
|
😂 |
* Add support for Firebolt Database (#5606) * Fixes issue #5622 (#5623) * Update Readme to reflect Firebolt data source (#5649) * Speed up BigQuery schema fetching (#5632) New method improves schema fetching by as much as 98% on larger schemas * Merge pull request from GHSA-vhc7-w7r8-8m34 * WIP: break the flask_oauthlib behavior * Refactor google-oauth to use cryptographic state. * Clean up comments * Fix: tests didn't pass because of the scope issues. Moved outside the create_blueprint method because this does not depend on the Authlib object. * Apply Arik's fixes. Tests pass. * Merge pull request from GHSA-g8xr-f424-h2rv * Merge pull request from GHSA-fcpv-hgq6-87h7 * Update changelog to incorporate security fixes and #5632 & #5606 (#5654) * Update changelog to incorporate security fixes and #5632 & #5606 * Added reference to sqlite fix * Bump to V10.1 * Missed package-lock.json on the first pass * Add a REDASH_COOKIE_SECRET for circleci * Revert "Add a REDASH_COOKIE_SECRET for circleci" This reverts commit 4576636. Moves config to the correct compose files * Move advocate to core requirements.txt file [debugging circleci failures] Co-authored-by: rajeshSigmoid <89909168+rajeshSigmoid@users.noreply.github.com> Co-authored-by: Aratrik Pal <44343120+AP2008@users.noreply.github.com> Co-authored-by: rajeshmauryasde <rajeshk@sigmoidanalytics.com> Co-authored-by: Katsuya Shimabukuro <katsu.generation.888@gmail.com>
The latest release (10.1) includes the Firebolt data source. Once you upgrade to 10.1 Firebolt becomes an option 👍 |
Wooo! Thanks @susodapop !! |
What type of PR is this? (check all applicable)
Description
Adding a query runner for firebolt
Related Tickets & Documents
Mobile & Desktop Screenshots/Recordings (if there are UI changes)