Andrushika opened a new pull request, #74403:
URL: https://github.com/apache/airflow/pull/74403

   ## Why
   
   #73464 added the task state store to the Java SDK with unit tests only. The 
Go SDK has an e2e test for the same API 
(`go_sdk_tests/test_go_sdk_task_state.py`). It runs a real task, reads the keys 
back through the REST API, and checks that `[state_store] 
default_retention_days` reaches the task process. Nothing checks this for Java 
yet, including that the coordinator passes the setting to the JVM.
   
   ## What
   
   Add a `RoundtripTaskState` task to `java-test-bundle` and a 
`TestJavaSDKTaskStateStore` class that follows the Go test. Java and Go modes 
now write the retention config through one helper, so the constant is renamed 
to `LANG_SDK_STATE_STORE_RETENTION_DAYS`. The retention test fails when the 
Java mode does not write the config (30 days instead of 7), and every test 
fails when the Java task throws before writing.
   
   related: #73464, #73420
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 5.5)
   
   Generated-by: Claude Code (Opus 5.5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to