Skip to content

Comments

refactor(!): Remove services lack of maintainers and users#6263

Merged
Xuanwo merged 1 commit intomainfrom
cleanup
Jun 4, 2025
Merged

refactor(!): Remove services lack of maintainers and users#6263
Xuanwo merged 1 commit intomainfrom
cleanup

Conversation

@Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Jun 4, 2025

Which issue does this PR close?

None

Rationale for this change

Code cleanup for opendal.

Remove services that lack maintainers and users.

These three services, atomicserver, icloud, and nebula_graph currently lack both maintainers and users. Despite this, we continue to invest significant effort to ensure they remain buildable.

Therefore, I propose that we remove them in our next major release. If there is renewed interest or user demand in the future, we can consider reintroducing them.

These services were selected based on their maintenance burden. In fact, more services such as dbfs and tikv could also be considered for removal. However, I will wait until #6262 has been resolved before taking further action.

What changes are included in this PR?

Remove services: atomicserver, icloud, and nebula_graph.

Are there any user-facing changes?

They can't use atomicserver, icloud, and nebula_graph anymore.

However, it's not actually broken, since they were never usable in the first place.

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Jun 4, 2025
@Xuanwo Xuanwo requested review from morristai and tisonkun June 4, 2025 10:37
@dosubot dosubot bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Jun 4, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 4, 2025
@Xuanwo Xuanwo merged commit 75908a9 into main Jun 4, 2025
358 of 359 checks passed
@Xuanwo Xuanwo deleted the cleanup branch June 4, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants