apache / apache/uniffle

[Bug] Adding server LocalStorageChecker HealthChecks fails to initialize

Open
#865 5 comments 3 reactions 1 assignee Claimed by @awdavidson View on GitHub
bug
Dominant language
Java
Stars
454
Forks
172
Avg merge
5d 17h
Merged PRs (30d)
5

Description

### Code of Conduct

- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

### Search before asking

- [X] I have searched in the [issues](https://github.com/apache/incubator-uniffle/issues?q=is%3Aissue) and found no similar issues.

### Describe the bug

Enabling health checks and configuring the LocalStorageChecker throws IllegalArgumentException on initialisation. I would like to limit the amount of local disk uniffle can consume, it seems that enabling health checks should help us achieve this, however, local storage health checks are current not working.

### Affects Version(s)

master

### Uniffle Server Log Output

_No response_

### Uniffle Engine Log Output

_No response_

### Uniffle Server Configurations

```yaml
rss.server.health.check.enable true
rss.server.health.checker.class.names org.apache.uniffle.server.LocalStorageChecker
```

### Uniffle Engine Configurations

_No response_

### Additional context

_No response_

### Are you willing to submit PR?

- [X] Yes I am 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.