You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by "JinwooHwang (via GitHub)" <gi...@apache.org> on 2025/10/23 16:55:44 UTC

[PR] Upgrade joda-time from 2.10.14 to 2.12.7 [geode]

JinwooHwang opened a new pull request, #7944:
URL: https://github.com/apache/geode/pull/7944

   ## Overview
   This PR upgrades the `joda-time` dependency from version 2.10.14 to 2.12.7, bringing in bug fixes, performance improvements, and updated timezone data.
   
   ## Changes Made
   
   ### Dependency Version Updates
   - **joda-time**: `2.10.14` → `2.12.7`
   
   ### Files Modified
   1. **build-tools/geode-dependency-management/src/main/groovy/org/apache/geode/gradle/plugins/DependencyConstraints.groovy**
      - Updated joda-time version constraint to 2.12.7
   
   2. **boms/geode-all-bom/src/test/resources/expected-pom.xml**
      - Updated expected joda-time version in BOM test resources
   
   3. **geode-assembly/src/integrationTest/resources/assembly_content.txt**
      - Updated JAR reference: `joda-time-2.10.14.jar` → `joda-time-2.12.7.jar`
   
   4. **geode-assembly/src/integrationTest/resources/gfsh_dependency_classpath.txt**
      - Updated classpath reference: `joda-time-2.10.14.jar` → `joda-time-2.12.7.jar`
   
   5. **geode-server-all/src/integrationTest/resources/dependency_classpath.txt**
      - Updated classpath reference: `joda-time-2.10.14.jar` → `joda-time-2.12.7.jar`
   
   ## Validation
   
   ### Build Validation
   All build tasks completed successfully:
   - ✅ `build`
   
   ### Test Validation
   - ✅ All tests passed with `gradlew test`
   
   ## Benefits
   - **Updated timezone data**: Latest timezone database for accurate date/time operations
   - **Bug fixes**: Includes multiple bug fixes from versions 2.10.15 through 2.12.7
   - **Performance improvements**: Enhanced performance in date/time calculations
   - **Security**: Latest stable version with all known issues addressed
   
   ## Compatibility
   - Fully backward compatible with existing code
   - No API changes affecting current usage
   - All existing tests pass without modification
   
   ## Release Notes References
   For detailed information about changes between versions, see:
   - [Joda-Time Release Notes](https://www.joda.org/joda-time/changes-report.html)
   
   ---
   
   **Note**: This upgrade is part of ongoing dependency maintenance to keep the project up-to-date with the latest stable versions of its dependencies.
   
   
   <!-- Thank you for submitting a contribution to Apache Geode. -->
   
   <!-- In order to streamline review of your contribution we ask that you
   ensure you've taken the following steps. -->
   
   ### For all changes, please confirm:
   - [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
   - [x] Has your PR been rebased against the latest commit within the target branch (typically `develop`)?
   - [x] Is your initial contribution a single, squashed commit?
   - [x] Does `gradlew build` run cleanly?
   - [ ] Have you written or updated unit tests to verify your changes?
   - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?
   


-- 
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: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Upgrade joda-time from 2.10.14 to 2.12.7 [geode]

Posted by "JinwooHwang (via GitHub)" <gi...@apache.org>.
JinwooHwang commented on PR #7944:
URL: https://github.com/apache/geode/pull/7944#issuecomment-3442439649

   Hi @sboorlagadda , all checks have passed. Thank you.


-- 
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: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Upgrade joda-time from 2.10.14 to 2.12.7 [geode]

Posted by "JinwooHwang (via GitHub)" <gi...@apache.org>.
JinwooHwang commented on PR #7944:
URL: https://github.com/apache/geode/pull/7944#issuecomment-3443036476

   Thank you so much for your support @raboof 


-- 
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: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Upgrade joda-time from 2.10.14 to 2.12.7 [geode]

Posted by "raboof (via GitHub)" <gi...@apache.org>.
raboof merged PR #7944:
URL: https://github.com/apache/geode/pull/7944


-- 
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: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org