close
Skip to content

[Improve][Transform] Update MetadataTransformConfig extends#9082

Merged
Hisoka-X merged 1 commit into
apache:devfrom
hailin0:hailin0-MetadataTransformConfig
Apr 1, 2025
Merged

[Improve][Transform] Update MetadataTransformConfig extends#9082
Hisoka-X merged 1 commit into
apache:devfrom
hailin0:hailin0-MetadataTransformConfig

Conversation

@hailin0

@hailin0 hailin0 commented Mar 31, 2025

Copy link
Copy Markdown
Member

Purpose of this pull request

[Transform] Update MetadataTransformConfig extends

Does this PR introduce any user-facing change?

No

How was this patch tested?

Check list

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the inheritance of MetadataTransformConfig by removing its dependency on ModelTransformConfig and instead implements Serializable.

  • Removed the extension of ModelTransformConfig.
  • Updated imports to reflect the change in class inheritance.
Comments suppressed due to low confidence (1)

seatunnel-transforms-v2/src/main/java/org/apache/seatunnel/transform/metadata/MetadataTransformConfig.java:26

  • Removing the extension from ModelTransformConfig might remove inherited functionality or configuration defaults relied upon elsewhere; please confirm that this change is intentional and that all necessary behavior is preserved.
public class MetadataTransformConfig implements Serializable {

@Hisoka-X Hisoka-X merged commit e9ab5d7 into apache:dev Apr 1, 2025
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.

4 participants