Uh oh!
There was an error while loading. Please reload this page.
fix(skills): update tsfile skills to follow current version - #916
Open
Alchuang22-dev wants to merge 3 commits into
Open
fix(skills): update tsfile skills to follow current version#916Alchuang22-dev wants to merge 3 commits into
Alchuang22-dev wants to merge 3 commits into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Updates the TsFile skill documentation/routing to reflect the current (2026-08-18) baseline and fix routing mismatches found in experiments.
Changes:
- Add new “current capabilities” reference docs (properties/TAG filters, encoding/compression, Java tools, DataFrame/Arrow, C++ build/write, advanced APIs).
- Refine skill scope and routing guidance between
tsfileandtsfile-cli. - Expand
docs-map.yamlauthorities and restructure tool routing entries.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/tsfile/references/properties-filters.md | New reference for file properties and Table TAG filter behavior across bindings. |
| skills/tsfile/references/performance.md | Adds pointer to the encoding/compression factory matrix reference. |
| skills/tsfile/references/java-tools.md | New reference for Java CLI tooling (import + point-count). |
| skills/tsfile/references/encoding-compression.md | New matrix reference for supported encoding/compression factories. |
| skills/tsfile/references/docs-map.yaml | Updates local authorities and adds “current_capabilities” and restructured tool routing. |
| skills/tsfile/references/dataframe-arrow.md | New reference for Python DataFrame and Arrow batch I/O. |
| skills/tsfile/references/cpp-build-write.md | New reference for C++ build/linkage and parallel write configuration. |
| skills/tsfile/references/advanced-apis.md | New reference for advanced APIs (row queries, metadata, recovery, TsBlock, etc.). |
| skills/tsfile/SKILL.md | Expands scope/routing rules and links to new reference docs. |
| skills/tsfile-cli/SKILL.md | Clarifies scope separation vs Java tools and point-count behavior. |
| cpp/tools/skills/tsfile-cli/SKILL.md | Aligns embedded skill doc with updated tsfile-cli scope and routing guidance. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| - cpp/tools/ | ||
| java: | ||
| owner_skill: | ||
| path: SKILL.md |
| ## Encoding and Compression | ||
| - Read `references/encoding-compression.md` for the current constructible |
| `cpp/src/cwrapper/tsfile_cwrapper.h`; do not guess buffer ownership. | ||
| The Java table point-count metadata is a reserved use of file properties. Use | ||
| `references/java-tools.md` for inspection or backfill instead of editing those |
| - Use `TsFileDataFrame` for a lazy unified numeric view across one or more | ||
| TsFile shards, metadata selection, or timestamp-aligned series access. | ||
| - Use Arrow batches for columnar interoperability without Python row loops. | ||
| - Use `references/java-tools.md` instead for Arrow file import through the Java |
Comment on lines
+32
to
+39
| | data type | current supported encodings | | ||
| |---|---| | ||
| | `BOOLEAN` | `PLAIN`, `RLE` | | ||
| | `INT32`, `INT64`, `DATE`, `TIMESTAMP` | `PLAIN`, `RLE`, `TS_2DIFF`, `GORILLA`, `ZIGZAG`, `CHIMP`, `SPRINTZ`, `RLBE` | | ||
| | `FLOAT` | `PLAIN`, `RLE`, `TS_2DIFF`, `GORILLA_V1`, `GORILLA`, `CHIMP`, `SPRINTZ`, `RLBE` | | ||
| | `DOUBLE` | the FLOAT set plus `CAMEL` | | ||
| | `TEXT`, `STRING` | `PLAIN`, `DICTIONARY` | | ||
| | `BLOB`, `OBJECT` | `PLAIN` | |
Comment on lines
+53
to
+60
| | value encoding | current accepted data types | | ||
| |---|---| | ||
| | `PLAIN` | all types handled by `PlainEncoder`/`PlainDecoder` | | ||
| | `DICTIONARY` | `STRING`, `TEXT` | | ||
| | `RLE` | `INT32`, `DATE`, `INT64`, `TIMESTAMP` | | ||
| | `TS_2DIFF`, `GORILLA` | `INT32`, `DATE`, `INT64`, `TIMESTAMP`, `FLOAT`, `DOUBLE` | | ||
| | `ZIGZAG` | `INT32`, `INT64` | | ||
| | `SPRINTZ` | `INT32`, `INT64`, `FLOAT`, `DOUBLE` | |
Alchuang22-devforce-pushed
the
feat/skills
branch
from
August 21, 2026 11:24
52d78c0 to
fe75640Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The PR is an update for tsfile skills.
Contents
The PR contains the following changes:
2026-02-19to2026-08-18