apache / apache/pinot

Move tests for specialized data structures such as TDigest, HLL, ThetaSketches into one test class

Open
#5,655 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
6.1k
Forks
1.5k
Avg merge
1d 21h
Merged PRs (30d)
189

Description

per @mayankshriv we should create one integration test for these specialized data structures such that:
1. The cluster is set up once at beginning.
2. For each of these features, we can setup the table -> run queries -> delete the table, iteratively

IT for theta sketches is already added in #5514. The other tests need to be moved to that class.

Contributor guide

Open the contributing guide

Research direction

Start with the ThetaSketch integration test added in #5514, then locate the existing TDigest and HLL integration tests. Run the relevant integration tests and compare their cluster setup and table lifecycle. Done means one test class sets up the cluster once and iteratively creates tables, runs queries, and deletes tables for each specialized data structure.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
databases, testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.