Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FLINK-22752] More robust state configurations in base flink-conf.yaml #257

Closed
wants to merge 1 commit into from

Conversation

tzulitai
Copy link
Contributor

@tzulitai tzulitai commented Aug 16, 2021

This PR intends to add a few configurations to the base flink-conf.yaml:

  • Activate local recovery. This includes setting a local directory for RocksDB to flush its sst / metadata files, as well as setting a local directory for recovery.
  • Activate partitioned indexes by default
  • Use more threads for transferring checkpoint files from DFS
  • Increase RocksDB compaction threads

MIrroring changes to official released Docker images

The changes here will be mirrored to our official released Docker images via this PR on apache/flink-statefun-docker: apache/flink-statefun-docker#14


Verifying the changes

The new configuration has been set in the flink-conf.yaml we use in the E2E tests as well. After doing that, I can see logs such as:

13:23:41,274 INFO org.apache.flink.statefun.e2e.smoke.SmokeRunner - 2021-08-16 05:23:41,278 INFO org.apache.flink.contrib.streaming.state.restore.RocksDBIncrementalRestoreOperation [] - Finished restoring from state handle: IncrementalLocalKeyedStateHandle{metaDataState=File State: file:/local/state/recovery/ ......

and

13:23:41,198 INFO org.apache.flink.statefun.e2e.smoke.SmokeRunner - 2021-08-16 05:23:41,202 INFO org.apache.flink.contrib.streaming.state.RocksDBKeyedStateBackendBuilder [] - Finished building RocksDB keyed state-backend at /local/state/rocksdb/ ......

which confirms that local recovery is working.

Also, other configurations have been picked up as well as confirmed by the logs.

Copy link
Contributor

@igalshilman igalshilman left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@tzulitai tzulitai closed this in 33abb7b Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants