Skip to content

Conversation

@luoyuxia
Copy link
Contributor

@luoyuxia luoyuxia commented Jun 11, 2025

Purpose

Linked issue: close #446

Brief change log

Tests

API and Format

Documentation

@luoyuxia luoyuxia force-pushed the remove-lake-cli branch 5 times, most recently from fb7c5c5 to fc434c8 Compare June 11, 2025 06:16
Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left one minor comment, and could you rebase to latest main branch ?

<exclude>com.alibaba.fluss.tools.ci.*</exclude>
<!-- exclude for fluss-dist -->
<exclude>com.alibaba.fluss.dist.DummyClass</exclude>
<exclude>com.alibaba.fluss.lakehouse.batch.ArrowRecordBatch</exclude>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why only add this class under path com.alibaba.fluss.lakehouse.* ?

@luoyuxia
Copy link
Contributor Author

@leonardBang Rebased done.

Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, CI passed, merging...

@leonardBang leonardBang merged commit 8424ba2 into apache:main Jun 11, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove fluss-lakehouse related modules after completing Fluss Lakehouse Storage refactor

2 participants