Skip to content

Initialize null handling in CompressedBigDecimalAggregatorTimeseriesTestBase to fix failing test#15252

Merged
abhishekagarwal87 merged 1 commit intoapache:masterfrom
LakshSingla:bd-initialize-module
Oct 25, 2023
Merged

Initialize null handling in CompressedBigDecimalAggregatorTimeseriesTestBase to fix failing test#15252
abhishekagarwal87 merged 1 commit intoapache:masterfrom
LakshSingla:bd-initialize-module

Conversation

@LakshSingla
Copy link
Contributor

@LakshSingla LakshSingla commented Oct 25, 2023 •
edited
Loading

Description

This PR initializes the CompressedBigDecimalAggregatorTimeseriesTestBase to fix the failing test in the master.

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@LakshSingla LakshSingla added this to the 28.0 milestone Oct 25, 2023
@abhishekagarwal87 abhishekagarwal87 merged commit 207398a into apache:master Oct 25, 2023
@LakshSingla LakshSingla mentioned this pull request Oct 25, 2023
Merged
LakshSingla added a commit to LakshSingla/druid that referenced this pull request Oct 25, 2023
@LakshSingla
Initialize null handling in CompressedBigDecimalAggregatorTimeseriesT…
f9b3c09 
…estBase to fix failing test(apache#15252)
LakshSingla added a commit that referenced this pull request Oct 25, 2023
@LakshSingla
Initialize null handling in CompressedBigDecimalAggregatorTimeseriesT…
f9e1212 
…estBase to fix failing test(#15252) (#15256)
CaseyPan pushed a commit to CaseyPan/druid that referenced this pull request Nov 17, 2023
@LakshSingla
Initialize null handling in CompressedBigDecimalAggregatorTimeseriesT…
7b2e65c 
…estBase to fix failing test(apache#15252)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abhishekagarwal87 abhishekagarwal87 abhishekagarwal87 approved these changes

Assignees

No one assigned

Labels

Development Blocker

Projects

None yet

Milestone

28.0

Development

Successfully merging this pull request may close these issues.

2 participants

@LakshSingla @abhishekagarwal87