Skip to content

Conversation

electrum
Copy link
Member

@electrum electrum commented Oct 2, 2025

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@electrum electrum requested review from dain and Copilot October 2, 2025 19:36
@cla-bot cla-bot bot added the cla-signed label Oct 2, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Extends aggregation annotation parsing to treat all implementation dependency annotations uniformly, enabling support for CastDependency without updating this parser each time a new dependency annotation is added.

  • Consolidated multiple explicit annotation filters into a single predicate using ImplementationDependency::isImplementationDependencyAnnotation
  • Removed now-unnecessary individual dependency annotation imports

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@electrum electrum merged commit 912c57e into trinodb:master Oct 2, 2025
189 of 190 checks passed
@github-actions github-actions bot added this to the 478 milestone Oct 2, 2025
@electrum electrum deleted the aggdep branch October 3, 2025 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants