apache / apache/polaris

Persistence model depends on public management API

Open
#780 4 comments 0 reactions 1 assignee Claimed by @gfakbar20 View on GitHub
bug
Dominant language
Java
Stars
2.1k
Forks
522
Avg merge
1d 17h
Merged PRs (30d)
137

Description

### Describe the bug

All entities of the persistence model have `asXyz()` functions to return a type from the management OpenAPI spec. While this is currently convenient, it introduces a cyclic dependency, which should generally be avoided:
* management-API depends on REST endpoints
* REST endpoint implementations depend on entities
* entities depend on management-API

The management API and its implementation should not leak into the persistence model.

### To Reproduce

_No response_

### Actual Behavior

_No response_

### Expected Behavior

_No response_

### Additional context

_No response_

### System information

_No response_

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.