Skip to content

Conversation

GreatEugenius
Copy link
Collaborator

Linked issue: #207

Purpose of change

Update the deployment introduction for Flink agents.

Tests

API

Documentation

Yes

Copy link
Contributor

@Sxnan Sxnan left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Please see my comments below.

@GreatEugenius
Copy link
Collaborator Author

Hi @Sxnan, thank you for the review! I have updated this PR; please review it again.

@GreatEugenius GreatEugenius requested a review from Sxnan October 6, 2025 07:17
address comments
@Sxnan
Copy link
Contributor

Sxnan commented Oct 6, 2025

Thanks for the update. LGTM.

@xintongsong Could you take a look at this PR?

Copy link
Contributor

@xintongsong xintongsong left a comment

Choose a reason for hiding this comment

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

LGTM

@xintongsong xintongsong merged commit 0fd0600 into apache:main Oct 7, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants