Repository navigation
release: 0.2.1: PyPI and crates.io metadata, pyarrow required, 429 retry, cron removed, RELEASING.md - #751
release: 0.2.1: PyPI and crates.io metadata, pyarrow required, 429 retry, cron removed, RELEASING.md#751techgopal wants to merge 30 commits into
Conversation
…, 429 retry for the crate upload step Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…ords, classifiers, project URLs Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…p, changelog) Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…hiv server, Spark Connect) Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…thub-only commit, unsourced speedup claim removed Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com> # Conflicts: # crates/krishiv-connectors-delta/Cargo.toml # crates/krishiv-session/Cargo.toml
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
Codex Review Summary: ❌ FailedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
@codex review |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
Merging this PR will not alter performance1 warning
Comparing Footnotes
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6c501a6266
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
… is from PyPI Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. What shall we delve into next? Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
…sion registration, deploy, check) Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…, gs://, az://); wheel test against MinIO; Data sources guide Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…out a pyarrow wheel (musllinux) does not build pyarrow from source Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c92d5fb04f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…s for the Krishiv wheel only Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
…(RS-15) Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…-031, PY-032) Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…, PY-032, SQL-1, PY-022, PY-006, RS-3, RS-15) Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com> # Conflicts: # crates/krishiv-cli/Cargo.toml
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
…ep pinned to 0.2.1; https is Spark's UNSUPPORTED_FILE_SYSTEM, not a wheel test case Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
…ered cfg branch) Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
wheel-sizes: what grew, and what could be cut
|
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
@coderabbitai review |
Action not completed
|
…a retry date; the offline cloud-store test runs without pyarrow (review of #751) Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
Review of 4b209a0 (Opus, ops/reviews/2026-11-15-sonnet-2-pr751-diff.md): no blockers. Fixed in 1685294: the crate-upload retry fell back to 60 s instead of 600 s when the 429 message had no date; the offline cloud-store test no longer skips the whole module without pyarrow (so musllinux runs it). For the owner (not changed): (1) pyarrow is now a required dependency and has no musllinux wheels, so |
Signed-off-by: gopal tathe <44522021+techgopal@users.noreply.github.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Packaging area, one PR for the 0.2.1 acceptance fixes in sonnet-2's lane. Issue ids (research/qa/2026-11-14-acceptance-issues.md):
cloudfeature (s3, gs, az, https, hf); Data sources guide;python/tests/test_cloud_stores.pycovers all five schemes.pyarrow>=16is a required dependency; the wheel test installs the wheel with--no-depsso musllinux (no pyarrow wheel) does not build pyarrow from source.telemetry-otlpfeature onkrishivandkrishiv-cli.Not in this PR: RS-1 (CLI install line, docs-3), PY-027/RS-22 (commit in build info, 0.2.2), RS-17 (empty features, owner).
Opus review: ops/reviews/2026-11-13-sonnet-2-release-021-diff.md. Open question B1 (pyarrow on musllinux) is answered by the wheel-test change; a musllinux user without a pyarrow wheel gets a source build on
pip install krishiv.