Skip to content

Conversation

claudiamurialdo
Copy link
Collaborator

Change GXExternalCollection to use a typed generic field internally, eliminating casts while preserving the existing public API.

…y, eliminating casts while preserving the existing public API.
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

fedeazzato
fedeazzato previously approved these changes Sep 25, 2025
… is unknown.

What we do know is the type of the wrapper SDT, which is T.
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests October 4, 2025 02:21 — with GitHub Actions Inactive
@claudiamurialdo claudiamurialdo temporarily deployed to kafka-integration-tests October 4, 2025 02:21 — with GitHub Actions Inactive
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo changed the title Change GXExternalCollection<T> to use typed internal field Minor improvement in understanding GxExternalCollection<T> Oct 4, 2025
@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests October 4, 2025 02:25 — with GitHub Actions Inactive
@claudiamurialdo claudiamurialdo temporarily deployed to kafka-integration-tests October 4, 2025 02:25 — with GitHub Actions Inactive
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo temporarily deployed to kafka-integration-tests October 4, 2025 02:27 — with GitHub Actions Inactive
@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests October 4, 2025 02:27 — with GitHub Actions Inactive
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

Copy link
Member

@fedeazzato fedeazzato left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks for the clear comments that explain why the inner list cannot be strong typed. 💯

@claudiamurialdo claudiamurialdo merged commit 18b8f7c into master Oct 9, 2025
11 of 12 checks passed
@claudiamurialdo claudiamurialdo deleted the fix/generic-external-collection branch October 9, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants