apache / apache/uniffle

[Improvement] Add parameter test for repartitionTest

Open
#817 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
Java
Stars
454
Forks
172
Avg merge
5d 17h
Merged PRs (30d)
5

Description

The repartition test takes too long time. So we use random parameter. If we can optimize the time, we should use parameter test to test all cases.

_Originally posted by @smallzhongfeng in https://github.com/apache/incubator-uniffle/pull/806#discussion_r1163709167_

Contributor guide

Open the contributing guide

Research direction

Start by locating the repartitionTest test and examine why its current random-parameter approach takes too long. Determine how parameterized coverage could include all cases without the current runtime cost, then verify that the test covers the intended cases within an acceptable duration.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.