[PR] [improve][build] Fix compile warnings in production code [pulsar]

["merlimat (via GitHub)" <gi...@apache.org>]

merlimat opened a new pull request, #25414:
URL: https://github.com/apache/pulsar/pull/25414

   ## Summary
   - Remove 11 redundant casts (Object, int, long, Position, KeyValue, X509Certificate[])
   - Add serialVersionUID to 62 Serializable classes missing it
   - Replace deprecated AccessController.doPrivileged with direct calls (4 locations)
   - Remove deprecated SecurityManager check in DnsResolverUtil
   - Fix 6 static method access warnings to use class references instead of instances
   - Fix fallthrough comment format in ManagedLedgerImpl switch statement
   
   ## Documentation
   - [ ] `doc-required`
   - [x] `doc-not-needed`
   - [ ] `doc`
   - [ ] `doc-complete`
   
   ## Matching PR in forked repositories
   - [ ] Matching PR in forked repository
   
   _This PR is part of a series to fix all compile warnings in production code._


-- 
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: commits-unsubscribe@pulsar.apache.org

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

Re: [PR] [improve][build] Fix compile warnings in production code [pulsar]

["merlimat (via GitHub)" <gi...@apache.org>]

merlimat merged PR #25414:
URL: https://github.com/apache/pulsar/pull/25414


-- 
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: commits-unsubscribe@pulsar.apache.org

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