apache / apache/pulsar

[Bug] JPMS support

Open
#20,853 7 comments 0 reactions 1 assignee Claimed by @tisonkun View on GitHub
Stale type/bug
Dominant language
Java
Stars
15.3k
Forks
3.8k
Avg merge
1d 14h
Merged PRs (30d)
160

Description

### Search before asking

- [ ] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.

### Version

3.0.0

### Minimal reproduce step

Try to use `pulsar-client-api` with JPMS.

### What did you expect to see?

A `module-info` file and support for JPMS in the client.

### What did you see instead?

```java
[WARN] **********************************************************************************************************************************************************************************************************
[WARN] * Required filename-based automodules detected: [pulsar-client-api-3.0.0.jar]. Please don't publish this project to a public artifact repository! *
[WARN] **********************************************************************************************************************************************************************************************************
```

### Anything else?

See https://github.com/apache/pulsar/issues/7971 - the issue was closed as stale but still takes effect, 3 years later.

I am willing to submit a PR (did check the box) but I'm not sure what is desired, and where to make the changes, so it's ideal if somebody with more experience in this project makes the changes. The main issue is if the client needs to retain Java 8 support - then the `module-info` needs to be built separately with either Java 9, or Java 11 (as it's the following LTS), so that JPMS can be supported alongside regular Java 8 classpath.

### Are you willing to submit a PR?

- [X] I'm willing to submit a PR!

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.