Repository navigation
Make LoggingToolset private and update Experimental notes in common.ai - #74694
Merged
Merged
Conversation
kaxil
reviewed
Oct 14, 2026
Lee-W
force-pushed
the
common-ai-stability-review
branch
from
October 14, 2026 09:09
c1aa356 to
343b703
Compare
Lee-W
marked this pull request as ready for review
October 14, 2026 10:11
Lee-W
requested review from
amoghrajesh,
ashb,
bugraoz93,
gopidesupavan,
jason810496,
jscheffl and
potiuk
as code owners
October 14, 2026 10:11
Lee-W
force-pushed
the
common-ai-stability-review
branch
from
October 14, 2026 10:13
343b703 to
a225862
Compare
kaxil
reviewed
Oct 14, 2026
Lee-W
force-pushed
the
common-ai-stability-review
branch
2 times, most recently
from
October 15, 2026 07:53
618e2ef to
103f825
Compare
kaxil
reviewed
Oct 16, 2026
Lee-W
force-pushed
the
common-ai-stability-review
branch
3 times, most recently
from
October 21, 2026 15:00
1659d13 to
dade363
Compare
kaxil
reviewed
Oct 21, 2026
Lee-W
force-pushed
the
common-ai-stability-review
branch
from
October 22, 2026 03:40
dade363 to
dccbafc
Compare
kaxil
approved these changes
Oct 22, 2026
kaxil
left a comment
kaxil
left a comment
Member
There was a problem hiding this comment.
LGTM, thanks. In operators/llm_sql.rst the new pin note sits ten lines above the existing install note about the same datafusion extra, so the pin could be one sentence in that note instead. Also, the last sentence of the PR description still says the Experimental notes stay on datasource_config and data_sources, which no longer matches the change. Neither needs to hold this up.
Lee-W
force-pushed
the
common-ai-stability-review
branch
from
October 23, 2026 13:13
dccbafc to
c54bca2
Compare
Member
|
Conflicts ! |
Lee-W
force-pushed
the
common-ai-stability-review
branch
from
October 25, 2026 02:00
c54bca2 to
3d7fecf
Compare
LoggingToolset is no longer exported from airflow.providers.common.ai.toolsets; tool-call logging stays available through enable_tool_logging and ToolLoggingCapability. The Experimental notes are removed from the parameters and classes that graduate to stable, and kept, with the concrete reason, on datasource_config and data_sources.
Moves managed agent toolsets, per-tool approval, ObjectStorageToolset, DecisionPolicy, pinned_arguments, llm_sql, llm_file_analysis, llm_schema_compare and the LangChain hook and bridge to the stable list, and gives every remaining experimental entry a concrete, checkable reason. Fixes the stale claim that CI does not run the Strands and ADK adapter tests.
Per-tool approval and min_confidence on a branch option are stable, so their docstrings no longer call them experimental. The toolset Layering section now describes ToolLoggingCapability as a capability that wraps the agent's toolset, the CI notes say Python source or test files to match the selective checks, and a long note in the llm_schema_compare guide is re-wrapped.
…en the framework CI job runs
The 1.0.0 note says LoggingToolset left the public API, so the Features bullet for apache#74449 should not list it as exported.
Outside canary runs the job overrides the CI image's packages and installs the newest framework release inside the exclude-newer window; canary runs (scheduled, push, manual and canary-labelled PRs) use each framework's own dependency pins.
1.0.0 is released and still exports LoggingToolset (apache#74449), so the removal note belongs under the next release. Restore LoggingToolset in the 1.0.0 export entry.
Notes for an unreleased version sit directly under the Changelog header; the release manager adds the version heading when cutting the release.
…usion pin as a dependency limit 1.0.0 shipped both parameters as stable, so the stability policy keeps them stable until the next major release. Drop their rows from the Experimental table and the Experimental notes from the operator docs.
The pin sat in its own note ten lines above the note about the datafusion extra it belongs to.
Lee-W
force-pushed
the
common-ai-stability-review
branch
from
October 26, 2026 06:21
3d7fecf to
04579a5
Compare
This was referenced Oct 29, 2026
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 free
to 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.
LoggingToolsetis no longer exported fromairflow.providers.common.ai.toolsets;tool-call logging stays available through
enable_tool_loggingandToolLoggingCapability. The Experimental notes are removed from the parametersand classes that graduate to stable, and kept, with the concrete reason, on
datasource_configanddata_sources.Was generative AI tooling used to co-author this PR?
Generated-by: [Claude] following the guidelines
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.