Skip to content

Update version to 0.7.0-dev6#3358

Merged
yingfeng merged 2 commits intoinfiniflow:mainfrom
qinling0210:update_version_0.7.0-dev6
May 6, 2026
Merged

Update version to 0.7.0-dev6#3358
yingfeng merged 2 commits intoinfiniflow:mainfrom
qinling0210:update_version_0.7.0-dev6

Conversation

@qinling0210
Copy link
Copy Markdown
Contributor

@qinling0210 qinling0210 commented May 6, 2026

What problem does this PR solve?

As title

Type of change

  • Python SDK impacted, Need to update PyPI
  • Other (please describe): Update version

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 6, 2026

📝 Walkthrough

Walkthrough

This PR bumps the Infinity SDK version from 0.7.0.dev5 to 0.7.0.dev6 across documentation, installation instructions, and version constants in Python, C++, and Go client implementations.

Changes

Version Bump to 0.7.0.dev6

Layer / File(s) Summary
Core Version Declaration
pyproject.toml
Package version updated from 0.7.0.dev5 to 0.7.0.dev6.
Client Version Constants
go/infinity.go, src/network/infinity_thrift_service_impl.cpp
Go ClientVersion constant and C++ Thrift service version mapping (index 36) updated to 0.7.0.dev6.
Client Implementation Annotations
client/cpp/infinity_client.cpp, python/infinity_sdk/infinity/remote_thrift/client.py, benchmark/remote_infinity/remote_query_benchmark.cpp, src/network/infinity_thrift_service.cppm
Client version comments and annotations updated to reflect 0.7.0.dev6; no functional changes.
Installation & Documentation
README.md, docs/getstarted/build_from_source.mdx, docs/getstarted/deploy_infinity_server.mdx, docs/getstarted/quickstart.mdx, example/README.md, python/infinity_sdk/README.md
SDK installation commands and version references updated across all documentation, guides, and examples.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 From dev5 to six, we bump with cheer,
Across all docs, both far and near,
Each version string now unified bright,
A consistent release, oh what delight! thump thump 🥕

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The description is minimal and vague. While it marks the Python SDK impact checkbox, it lacks substantive details about the changes, rationale, and scope of the version bump. Expand the description to explain why this version bump is needed, what changes are included in dev6, and confirm all affected components are accounted for.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: bumping the project version from 0.7.0-dev5 to 0.7.0-dev6 across all files.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 golangci-lint (2.12.1)

level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix . does not contain main module or its selected dependencies"


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@qinling0210 qinling0210 self-assigned this May 6, 2026
@qinling0210 qinling0210 added the ci PR can be test label May 6, 2026
@qinling0210 qinling0210 marked this pull request as ready for review May 6, 2026 11:59
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
client/cpp/infinity_client.cpp (1)

28-28: ⚡ Quick win

Centralize the client-version index to avoid future drift.

36 is duplicated across multiple components. Consider sourcing this from a shared generated constant (or version manifest) so future bumps don’t desync client/server/docs.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@client/cpp/infinity_client.cpp` at line 28, The literal client version index
passed to request.__set_client_version(36) must be replaced with a shared
generated constant to avoid drift; create or use a central constant (e.g.,
CLIENT_VERSION_INDEX or CLIENT_VERSION) in a generated header/manifest and
include it in this translation unit, then call
request.__set_client_version(CLIENT_VERSION_INDEX) instead of the hard-coded 36;
ensure the constant’s name matches other components’ usage so future bumps
update a single source of truth.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@client/cpp/infinity_client.cpp`:
- Line 28: The literal client version index passed to
request.__set_client_version(36) must be replaced with a shared generated
constant to avoid drift; create or use a central constant (e.g.,
CLIENT_VERSION_INDEX or CLIENT_VERSION) in a generated header/manifest and
include it in this translation unit, then call
request.__set_client_version(CLIENT_VERSION_INDEX) instead of the hard-coded 36;
ensure the constant’s name matches other components’ usage so future bumps
update a single source of truth.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3133d441-c26e-4f01-94af-ef125beae413

📥 Commits

Reviewing files that changed from the base of the PR and between c2e21a0 and 1170c60.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (13)
  • README.md
  • benchmark/remote_infinity/remote_query_benchmark.cpp
  • client/cpp/infinity_client.cpp
  • docs/getstarted/build_from_source.mdx
  • docs/getstarted/deploy_infinity_server.mdx
  • docs/getstarted/quickstart.mdx
  • example/README.md
  • go/infinity.go
  • pyproject.toml
  • python/infinity_sdk/README.md
  • python/infinity_sdk/infinity/remote_thrift/client.py
  • src/network/infinity_thrift_service.cppm
  • src/network/infinity_thrift_service_impl.cpp

@yingfeng yingfeng merged commit 39c4734 into infiniflow:main May 6, 2026
88 of 89 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci PR can be test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants