Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SBT support for project #1383

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

Add SBT support for project #1383

wants to merge 9 commits into from

Conversation

EmilyMatt
Copy link
Contributor

@EmilyMatt EmilyMatt commented Feb 10, 2025

Which issue does this PR close?

Closes #1344

Rationale for this change

Should optimize incremental compilation and allow for much faster development and tests cycles.
Modernizes the work environment.

What changes are included in this PR?

Simply added a build.sbt file, plugins.sbt file. and updated some docs.

How are these changes tested?

Simply by running "sbt assembly", or with added profiles "sbt assembly -Dspark3.4 -Dscala2.12 -DLinux-amd64", or whichever ones you use.

EmilyFlarionIO and others added 9 commits February 9, 2025 16:22
## Which issue does this PR close?

## Rationale for this change

Current Comet unified memory pool is a greedy pool. One thread (consumer) can take a large amount of memory that causes OOM for other threads, especially for aggregation.

## What changes are included in this PR?

Added a fair version of unified memory pool similar to DataFusion `FairSpilPool` that caps the memory usage at `pool_size/num`
The fair unified memory pool is the default for off-heap mode with this PR

## How are these changes tested?

Exisiting tests
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.

Suggestion: Move to SBT for faster development cycle?
3 participants