Skip to content

feat(SqlLab): Ai Assist for SQL Lab #34535

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 269 commits into
base: master
Choose a base branch
from
Draft

feat(SqlLab): Ai Assist for SQL Lab #34535

wants to merge 269 commits into from
+5,468 −161

Conversation

alexandrusoare
Copy link
Contributor

SUMMARY

AI Assist for SQL Lab

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

mistercrunch and others added 30 commits February 12, 2025 11:40
@mistercrunch @michael-s-molina
fix: set Rich tooltip -> 'Show percentage' to false by default (apa…
699d22c 
…che#32212)

(cherry picked from commit d3b854a)
@justinpark @michael-s-molina
fix(sqllab): close the table tab (apache#32224)
8b1de4c 
(cherry picked from commit 937d40c)
@LevisNgigi @michael-s-molina
fix(Scope): Correct issue where filters appear out of scope when sort…
c6e764e 
… is unchecked. (apache#32115)

(cherry picked from commit af3589f)
@dpgaspar @michael-s-molina
chore(ci): fix ephemeral env null issue number (apache#32220)
d0417c3 
(cherry picked from commit 60424c4)
@dpgaspar @michael-s-molina
chore(ci): fix ephemeral env null issue number (v2) (apache#32221)
6b95ecd 
(cherry picked from commit 31d6f5a)
@fardin-developer @michael-s-molina
fix: Update 'Last modified' time when modifying RLS rules (apache#32227)
e90b359 
Co-authored-by: Fardin Mustaque <[email protected]>
(cherry picked from commit 52563d3)
@mistercrunch @michael-s-molina
docs: adding notes about using uv instead of raw pip (apache#32239)
adce138
@eschutho @michael-s-molina
fix: remove sort values on stacked totals (apache#31333)
39859e0 
(cherry picked from commit 15fbb19)
@gpchandran @michael-s-molina
fix: upgrade to 3.11.11-slim-bookworm to address critical vulnerabili…
4a298e8 
…ties (apache#32240)

(cherry picked from commit ad05732)
@tahvane1 @michael-s-molina
fix(Datasource): handle undefined datasource_type in fetchSyncedColum…
ed217ce 
…ns (apache#32218)

Co-authored-by: Erkka Tahvanainen <[email protected]>
(cherry picked from commit 9da3095)
@michael-s-molina
fix: Decimal values for Histogram bins (apache#32253)
c7cc436 
(cherry picked from commit ffe9244)
@hainenber @michael-s-molina
fix(viz/table): selected column not shown in Conditional Formatting p…
1463111 
…opover (apache#32272)

Signed-off-by: hainenber <[email protected]>
(cherry picked from commit dcc9628)
@US579 @michael-s-molina
fix: keep the tab order (apache#30888)
45045d3 
Co-authored-by: Steven Liu <[email protected]>
(cherry picked from commit d5a5bd4)
@kgabryje @michael-s-molina
fix: Download as PDF fails due to cache error (apache#32332)
2c03455 
(cherry picked from commit 42a3c52)
@aurokk @michael-s-molina
fix: oauth2 trino (apache#31993)
e8246ea 
(cherry picked from commit 7ce1a34)
@eschutho @michael-s-molina
fix: revert "fix: remove sort values on stacked totals (apache#31333)" (
b2bd39c 
apache#32337)

(cherry picked from commit 422a07b)
@hainenber @michael-s-molina
fix(fe/dashboard-list): display modifier info for Last modified data (
7aa4cd4 
apache#32035)

Signed-off-by: hainenber <[email protected]>
(cherry picked from commit 88cf2d5)
@LevisNgigi @michael-s-molina
fix(roles): Add SqlLabPermalinkRestApi as default sqlab roles. (apach…
724f2dc 
…e#32284)

(cherry picked from commit 2c37ddb)
@EnxDev @michael-s-molina
fix(SSHTunnelForm): make the password tooltip visible (apache#32356)
7ef38fe 
(cherry picked from commit 4c4b5e8)
@betodealmeida @michael-s-molina
fix(firebolt): allow backslach escape for single quotes (apache#32350)
3eda222 
(cherry picked from commit 22fe985)
@vedantprajapati @michael-s-molina
fix(docker): Configure nginx for consistent port mapping and hot relo…
98c82ad 
…ading (apache#32362)

(cherry picked from commit 0f07d78)
@maybedino @michael-s-molina
fix: clickhouse-connect engine SSH parameter (apache#32348)
09a93d2 
(cherry picked from commit 8dcae81)
@betodealmeida @michael-s-molina
fix: ensure metric_macro expands templates (apache#32344)
654062d
@dpgaspar @michael-s-molina
fix: bump FAB to 4.5.4 (apache#32325)
ae3493a 
(cherry picked from commit c02a0a0)
@yuribogomolov @michael-s-molina
fix(pinot): revert join and subquery flags (apache#32382)
d611c25 
(cherry picked from commit 822d72c)
@DamianPendrak @michael-s-molina
fix(viz): update nesting logic to handle multiple dimensions in Parti…
b11fd2f 
…tionViz (apache#32290)

(cherry picked from commit 6317a91)
@codenamelxl @michael-s-molina
fix(clickhouse): get_parameters_from_uri failing when secure is true (a…
8a09fd9 
…pache#32423)

(cherry picked from commit 84b52b2)
@hainenber @michael-s-molina
fix(com/grid-comp/markdown): pin remark-gfm to v3 to allow inline c…
f129139 
…ode block by backticks in Markdown (apache#32420)

Signed-off-by: hainenber <[email protected]>
@villebro @michael-s-molina
fix(plugin-chart-echarts): remove erroneous upper bound value (apache…
92549ef 
…#32473)

(cherry picked from commit 5766c36)
@rusackas @michael-s-molina
fix(tooltip): displaying <a> tags correctly (apache#32488)
6dd10ab 
(cherry picked from commit 6c3886a)
luizotavio32 and others added 19 commits June 9, 2025 08:56
@luizotavio32 @michael-s-molina
fix: Migrate charts with empty query_context (apache#33710)
e83ac1d 
(cherry picked from commit 08655a7)
@withnale @michael-s-molina
feat(api): Added uuid to list api calls (apache#32414)
e5c6d3e 
(cherry picked from commit 8decc9e)
@Prokos @michael-s-molina
fix(template_processing): get_filters now works for IS_NULL and IS_NO…
94369ca 
…T_NULL operators (apache#33296)

(cherry picked from commit cc34608)
@rad-pat @michael-s-molina
fix(dashboard): show dashboard thumbnail images when retrieved (apach…
77d1849 
…e#33726)
@Quatters @michael-s-molina
fix(Alerts & reports): invalid "Last updated" time formatting (apache…
be13950 
…#33719)

(cherry picked from commit 3ef92e5)
@Quatters @michael-s-molina
fix(explore): add gap to the "Cached" button (apache#33717)
24ec0c8
@rad-pat @michael-s-molina
fix(dataset): Fix plural toast messages (apache#33743)
cbbf335 
(cherry picked from commit a64b9ac)
@anantaoutlook @michael-s-molina
fix: sync dot color between dashboard chart and edit chart (apache#33748
715fe6f 
)

(cherry picked from commit 78ad6db)
@Quatters @michael-s-molina
fix(create chart page): add missing space between words (apache#33752)
4f1ea35
@betodealmeida @michael-s-molina
fix: apply d3 format to BigNumber(s) (apache#33759)
cdbdaf4 
(cherry picked from commit d7d7b7c)
@msyavuz @michael-s-molina
fix(tooltip): Sanitize tooltip html (apache#33765)
ad10abd 
(cherry picked from commit 8fd0fd6)
@gpchandran @michael-s-molina
build: update Dockerfile to 3.11.13-slim-bookworm (apache#33745)
cff2ce0 
(cherry picked from commit 7b6885a)
@amaannawab923 @nelsondrew @michael-s-molina
fix(Echarts): Echarts Legend Scroll fix (apache#33779)
9a6261b 
Co-authored-by: Amaan Nawab <[email protected]>
@michael-s-molina
fix: SQL Lab warning message sizes (apache#33817)
f906adb 
(cherry picked from commit 775a702)
@michael-s-molina
chore: Adds RC4 data to CHANGELOG.md
465e2a9
@awilliamsTT
Fix issues with creating datasets
073213f
@awilliamsTT
Bump the max row limit in SQL Lab to 100000
37bc5a4
@diegoscarabelli
Merge tag '5.0.0' into awilliamsTT/ai-assistant
05d6f74
@alexandrusoare
merging
5b0537a
@korbit-ai Korbit AI
Copy link

korbit-ai bot commented Aug 4, 2025

Based on your review schedule, I'll hold off on reviewing this PR until it's marked as ready for review. If you'd like me to take a look now, comment /korbit-review.

Your admin can change your review schedule in the Korbit Console

@github-actions github-actions bot added i18n Namespace | Anything related to localization risk:db-migration PRs that require a DB migration api Related to the REST API doc Namespace | Anything related to documentation dependencies:npm packages i18n:traditional-chinese labels Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sadpandajoe sadpandajoe Awaiting requested review from sadpandajoe sadpandajoe will be requested when the pull request is marked ready for review sadpandajoe is a code owner

@geido geido Awaiting requested review from geido geido will be requested when the pull request is marked ready for review geido is a code owner

@eschutho eschutho Awaiting requested review from eschutho eschutho will be requested when the pull request is marked ready for review eschutho is a code owner

@rusackas rusackas Awaiting requested review from rusackas rusackas will be requested when the pull request is marked ready for review rusackas is a code owner

@betodealmeida betodealmeida Awaiting requested review from betodealmeida betodealmeida will be requested when the pull request is marked ready for review betodealmeida is a code owner

@mistercrunch mistercrunch Awaiting requested review from mistercrunch mistercrunch will be requested when the pull request is marked ready for review mistercrunch is a code owner

@michael-s-molina michael-s-molina Awaiting requested review from michael-s-molina michael-s-molina will be requested when the pull request is marked ready for review michael-s-molina is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
api Related to the REST API dependencies:npm doc Namespace | Anything related to documentation hold:sip! i18n:traditional-chinese i18n Namespace | Anything related to localization packages review:draft risk:db-migration PRs that require a DB migration size/XXL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.