MarvinCai opened a new issue #12271: URL: https://github.com/apache/pulsar/issues/12271
As discussed in dev mail list: https://lists.apache.org/thread.html/re99b342765b2cfb26f3da0eedfb78e7312de158bf6c7ed0085d32781%40%3Cdev.pulsar.apache.org%3E We want to introduce Maven [Modernizer](https://github.com/gaul/modernizer-maven-plugin) plugin to Pulsar. This plugin helps detecting uses of legacy APIs which can be replaced with modern APIs that are often more performant, safer, and idiomatic than the legacy equivalents. We'll break down the task by sub-module so anyone interested can work on a sub-module independently. We'll start with checking violation with **Java 8**. Acceptance criteria will be **passing existing CI**. Help are welcome, please reply with module you want to work on so there won't be conflict. And if change for single module is too big, please break down pr to reasonable size, say < 100 change so it's easier to review. **Please don't close the master issue via comment.** ``` <plugin> <groupId>org.gaul</groupId> <artifactId>modernizer-maven-plugin</artifactId> <version>2.3.0</version> <configuration> <failOnViolations>true</failOnViolations> <javaVersion>8</javaVersion> </configuration> <executions> <execution> <id>modernizer</id> <goals> <goal>modernizer</goal> </goals> </execution> </executions> </plugin> ``` - [ ] managed-ledger - [ ] pulsar-common - [ ] pulsar-broker-common - [ ] pulsar-broker-auth-athenz - [ ] pulsar-broker-auth-sasl - [ ] pulsar-broker - [ ] pulsar-client-api - [ ] pulsar-client - [ ] pulsar-client-admin-api - [ ] pulsar-client-admin - [ ] pulsar-client-tools - [ ] pulsar-client-tools-test - [ ] pulsar-client-auth-athenz - [ ] pulsar-client-auth-sasl - [ ] pulsar-websocket - [ ] pulsar-proxy - [ ] pulsar-discovery-service - [ ] pulsar-zookeeper-utils - [ ] pulsar-testclient - [ ] pulsar-broker-auth-sasl - [ ] pulsar-client-auth-sasl - [ ] pulsar-config-validation - [ ] pulsar-transaction - [ ] pulsar-functions - [ ] pulsar-metadata - [ ] pulsar-package-management -- 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]
