Skip to content

[doc] use standard Flink docker image in QuickStart#1997

Merged
luoyuxia merged 8 commits into
apache:mainfrom
xx789633:flink-user
Dec 5, 2025
Merged

[doc] use standard Flink docker image in QuickStart#1997
luoyuxia merged 8 commits into
apache:mainfrom
xx789633:flink-user

Conversation

@xx789633
Copy link
Copy Markdown
Contributor

Purpose

Linked issue: close #xxx

Brief change log

Tests

API and Format

Documentation

Comment thread website/docs/quickstart/flink.md Outdated
2. Create a `docker-compose.yml` file with the following content:
2. Create a Dockerfile named `fluss.Dockerfile` as follows. You can adjust the Flink version as needed. Please make sure to download the compatible versions of [fluss-flink connector jar](/downloads), [flink-connector-faker](https://github.com/knaufk/flink-faker/releases) in the Dockerfile.

```Dockerfile
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.

A docker file looks wried to me. Could we just mount the libs to flink image just like we did for iceberg?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done.

Copy link
Copy Markdown
Contributor

@luoyuxia luoyuxia left a comment

Choose a reason for hiding this comment

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

@xx789633 Thanks for the pr. Left some comment

@luoyuxia
Copy link
Copy Markdown
Contributor

Also change Building a Streaming Lakehouse page

#begin Flink cluster
jobmanager:
image: apache/fluss-quickstart-flink:1.20-$FLUSS_DOCKER_VERSION$
image: flink:${FLINK_VERSION}
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.

IIUC, with ${FLINK_VERSION}, users can't directly copy & paste the yaml?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, but this saves effort to set the flink version for each component.

Copy link
Copy Markdown
Contributor

@luoyuxia luoyuxia left a comment

Choose a reason for hiding this comment

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

+1

@luoyuxia luoyuxia merged commit 966ca22 into apache:main Dec 5, 2025
2 checks passed
LiebingYu pushed a commit to LiebingYu/fluss that referenced this pull request Dec 5, 2025
Ugbot pushed a commit to Ugbot/fluss that referenced this pull request Apr 26, 2026
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.

2 participants