Skip to content

Conversation

@wenjin272
Copy link
Collaborator

@wenjin272 wenjin272 commented Jan 22, 2026

Linked issue: #393

Purpose of change

  1. Add flink-agents-runtime-bundle module to decrease the complexity for users to introduce execution dependencies in flink-agents project.
  2. Add correspond documents, based on Add documentation for adding Maven dependencies #399

Tests

No

API

No

Documentation

  • doc-needed
  • doc-not-needed
  • doc-included

@github-actions github-actions bot added priority/major Default priority of the PR or issue. fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. doc-included Your PR already contains the necessary documentation updates. labels Jan 22, 2026
@wenjin272 wenjin272 force-pushed the doc-maven branch 2 times, most recently from 5872828 to 3897ac2 Compare January 23, 2026 06:10
</parent>

<artifactId>flink-agents-ide-support</artifactId>
<name>Flink Agents : IDE Support</name>
Copy link
Contributor

@xintongsong xintongsong Jan 23, 2026

Choose a reason for hiding this comment

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

Let's add a <description> here to explain what this artifact is for. Otherwise, it's a bit implicit.


**Required for developing Agent applications:**
- `flink-agents-api` - Core API for Flink-Agents
- `flink-streaming-java` and `flink-table-api-java` - Core API for Flink
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- `flink-streaming-java` and `flink-table-api-java` - Core API for Flink
- `flink-streaming-java` and/or `flink-table-api-java` - Core API for Flink


**Additional dependencies for running in IDE:**
- `flink-agents-ide-support` - Runtime execution dependencies (required for local execution/testing).

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's use "required" vs. "optional" here.

We should explain that, unlike running in a Flink cluster, when running in IDE, runtime dependencies are absent, so we need this additional dependency, and use "run with provided dependencies" feature of IDE.

And we should emphasis to use provided scope.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-included Your PR already contains the necessary documentation updates. fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants