JinwooHwang commented on code in PR #7940:
URL: https://github.com/apache/geode/pull/7940#discussion_r2442505608


##########
TESTING.md:
##########
@@ -13,8 +13,10 @@ Tests are broken up into five types - unit, integration, 
distributed, acceptance
   `./gradlew distributedTest`
 * Acceptance tests: test Geode from end user perspective  
   `./gradlew acceptanceTest`
-* Upgrade tests: test compatibility between versions of Geode and rolling 
upgrades  
+* Upgrade tests: test backwards compatibility and rolling upgrades between 
versions of Geode  
   `./gradlew upgradeTest`
+  
+  **Note**: Rolling upgrades are **NOT supported** across the Jakarta EE 10 
migration boundary (pre-migration → post-migration) for Tomcat session 
replication due to the javax.servlet → jakarta.servlet API incompatibility. 
Rolling upgrades within the same API era continue to work.

Review Comment:
   Thank you for pointing it out, @sboorlagadda. We should definitely document 
this breaking change clearly.
   
   We can add a migration guide section that includes:
   
   **Version requirements**
   Tomcat 10.1+ and 11.x require Apache Geode 2.x.
   
   **Deprecation timeline**
   Once we sunset Apache Geode 1.x, support for Tomcat 6/7/8/9 will be 
completely removed.
   
   This will give users a clear upgrade path and help them plan ahead. I can 
draft an initial update for the release notes and migration guide to make these 
requirements explicit.



-- 
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