Skip to content

Feat starrocks olap connector test #8482

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
k-anshul wants to merge 24 commits into
base: main
Choose a base branch
from
Draft

Feat starrocks olap connector test #8482

k-anshul wants to merge 24 commits into from

Conversation

@k-anshul
Copy link
Member

@k-anshul k-anshul commented Dec 11, 2025

INSERT DESCRIPTION HERE

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@EdwardArchive
only olap connector(not incloud model)
fbde1a6
@EdwardArchive
fix : message "not available for dialect 'starrocks'" & starrocks icon
ad513a0
@EdwardArchive
feat(starrocks): implement histogram and timeseries support
aee42f2 
- Add StarRocks support for ColumnRugHistogram with generate_series()
- Add StarRocks support for ColumnNumericHistogram (FD & Diagnostic methods)
- Add CTE-based ColumnTimeseries for StarRocks (no TEMP TABLE required)
- Add helper functions: rangeNumbersEnd(), isNonNullFinite() for StarRocks
- Handle reserved keywords (values, range) with backtick escaping
@EdwardArchive
feat(starrocks): add support for resolving timestamps in StarRocks di…
692f7c6 
…alect
@EdwardArchive
feat(starrocks): enhance StarRocks connector with improved context ha…
4c09870 
…ndling, connection timeouts, and error messages
@EdwardArchive
feat(starrocks): add StarRocks form and error handling to AddDataForm
c47dbd9
@EdwardArchive
Update docs
0a3dbc5
@EdwardArchive
fix: update code based on PR review feedback
ed2301b 
- addressed multiple review comments
- cleaned up import and minor issues
@EdwardArchive
fix: apply fixes from PR comments
ffdce1b
@EdwardArchive
fix: resolve build errors
948fa2a 
- Remove unused 'strings' import from starrocks.go
- Fix undefined variable 't' in olap.go StarRocks case
- Update date_trunc comment to reference validation location
@EdwardArchive
fix: update reserved keyword handling and SQL name sanitization for S…
4de5c64 
…tarRocks dialect
@EdwardArchive
fix: improve dry run handling in Query method and simplify SQL min query
2987351
@EdwardArchive
fix: correct safeName call in Druid maxSQL query
003001e
@EdwardArchive
fix: use equality check instead of Before for StarRocks time range bins
ae95f12
@EdwardArchive
fix: resolve build errors and metrics context propagation in Data Exp…
436fa8c 
…lorer

- Fix build compilation errors
- Ensure selected catalog (Database), schema (DatabaseSchema), and table
  are properly propagated when creating metrics from Data Explorer
@EdwardArchive
change test code and fix on AddStarRocksFrom
b83c569
@EdwardArchive
Apply minor fixes from code review comments
f4cd71f
@EdwardArchive
Refactor: removed unused code as suggested by Git CI
0d75a03
@EdwardArchive
fix: Add only for the StarRocks compatibility by executor part
60c6614
@EdwardArchive
fix change c.getDB to c.db
6cd5488
@EdwardArchive
fix: simplify database connection retrieval in StarRocks driver
11db47c
@EdwardArchive
fix: update column handling for StarRocks compatibility in numeric an…
193ba9c 
…d rug histograms + build check
@EdwardArchive
feat: enhance error handling and form structure in AddStarRocksForm c…
3bec873 
…omponent
@EdwardArchive
fix the failing checks.
3f61433
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@k-anshul @EdwardArchive