Skip to content

Update AI resource skills#9696

Merged
nishantmonu51 merged 5 commits into
mainfrom
rohithreddykota/update-ai-resource-skills
Jul 17, 2026
Merged

Update AI resource skills#9696
nishantmonu51 merged 5 commits into
mainfrom
rohithreddykota/update-ai-resource-skills

Conversation

@rohithreddykota

@rohithreddykota rohithreddykota commented Jul 16, 2026

Copy link
Copy Markdown
Member

Adds resource-authoring guidance to the AI development skills (runtime/ai/instructions/data/resources/).

  • connector: StarRocks, Databricks, and DuckLake connectors; Gemini AI provider; live-OLAP capability list
  • metrics_view: tags on dimensions/measures; rollups
  • model: glob start/end/last partition window; Iceberg/Delta reads via DuckDB
  • canvas: tab groups
  • rill.yaml: ai_connector; resource-level tags

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!

Add resource-authoring guidance for features shipped since the skills were last updated, verified against the parser/driver code:

- connector: StarRocks, Databricks, DuckLake, and Gemini connectors; live-OLAP capability list
- metrics_view: tags on dimensions/measures; rollups
- model: glob start/end/last partition window; Iceberg/Delta reads via DuckDB
- canvas: tab groups
- rill.yaml: ai_connector; resource-level tags
@rohithreddykota rohithreddykota changed the title Update AI resource skills for 0.79-0.88 features Update AI resource skills Jul 16, 2026

@nishantmonu51 nishantmonu51 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall LGTM. @begelundmuller : As you are working in this area, would be good for you to review as well.

Comment thread runtime/ai/instructions/data/resources/connector.md Outdated
Comment thread runtime/ai/instructions/data/resources/connector.md Outdated
@nishantmonu51

Copy link
Copy Markdown
Collaborator

@codex: rveiew

@chatgpt-codex-connector

Copy link
Copy Markdown

Summary

  • No new changes were needed: the requested review comments are already addressed in the current branch. The also wording is absent, and the OLAP capability line matches the suggested wording. runtime/ai/instructions/data/resources/connector.mdL17
  • I did not create a commit or PR because the working tree had no changes, and the instructions prohibit calling make_pr without committed changes.

Testing

  • nl -ba runtime/ai/instructions/data/resources/connector.md | sed -n '12,24p'
  • git status --short

View task →

Comment thread runtime/ai/instructions/data/resources/metrics_view.md Outdated
@pjain1

pjain1 commented Jul 17, 2026

Copy link
Copy Markdown
Member

made few changes, here's my review - #9697

* review

* format

@begelundmuller begelundmuller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread runtime/ai/instructions/data/resources/rillyaml.md Outdated
@nishantmonu51
nishantmonu51 merged commit a97bffd into main Jul 17, 2026
7 checks passed
@nishantmonu51
nishantmonu51 deleted the rohithreddykota/update-ai-resource-skills branch July 17, 2026 07:50
nishantmonu51 pushed a commit that referenced this pull request Jul 17, 2026
* Update AI resource skills for 0.79-0.88 features

Add resource-authoring guidance for features shipped since the skills were last updated, verified against the parser/driver code:

- connector: StarRocks, Databricks, DuckLake, and Gemini connectors; live-OLAP capability list
- metrics_view: tags on dimensions/measures; rollups
- model: glob start/end/last partition window; Iceberg/Delta reads via DuckDB
- canvas: tab groups
- rill.yaml: ai_connector; resource-level tags

* Remove redundant 'also' from OLAP capability line

* Apply suggestion from @rohithreddykota

* review (#9697)

* review

* format

* Update runtime/ai/instructions/data/resources/rillyaml.md

---------

Co-authored-by: Parag Jain <pjain1@apache.org>
Co-authored-by: Benjamin Egelund-Müller <b@egelund-muller.com>
(cherry picked from commit a97bffd)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants