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.

Sign up for GitHub

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

Jump to bottom

Improvement to have specify scaling factor for timestamp column used in partitioning by configuration #160

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Improvement to have specify scaling factor for timestamp column used in partitioning by configuration #160

wants to merge 1 commit into from

Conversation

hariprasad-k
Copy link

Problem

#126 Improve to storage partitioning scheme to be flexible and configurable at runtime

Solution

This provides ability to specify scaling factor for timestamp column used in partitioning by configuration

Does this solution apply anywhere else?
  • yes
  • no
If yes, where?

Test Strategy

Testing done:
  • Unit tests
  • Integration tests
  • System tests
  • Manual tests

Release Plan

@hariprasad-k
Merge
b06c1b1 
Unit tests

Unit test

Dummy

Removed dummy
@hariprasad-k hariprasad-k requested a review from a team as a code owner November 3, 2020 17:05
@ghost
Copy link

ghost commented Nov 3, 2020

@confluentinc It looks like @hariprasad-k just signed our Contributor License Agreement. 👍

Always at your service,

clabot

@hariprasad-k
Copy link
Author

The patch also fixes #121 too

@hariprasad-k hariprasad-k mentioned this pull request Nov 3, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hariprasad-k