Skip to content

Merge getUsedSegmentsForInterval and getUsedSegmentAndCreatedDateForInterval into one method #5775

Closed

Description

@zhangxinyu1

see #5556

Activity

  1. zhangxinyu1 commented on Jun 28, 2018

    @zhangxinyu1
    ContributorAuthor

    Hi @gianm,
    I want to merge the method getUsedSegmentsForInterval() and getUsedSegmentAndCreatedDateForInterval() of IndexerMetadataStorageCoordinator.java into one method. There is only one difference between them: The former does not include information of created_date, but the latter does. I don't know why created_date is not a part of DataSchema. Could we make DataSchema include created_date?

  2. stale commented on Jun 21, 2019

    @stale

    This issue has been marked as stale due to 280 days of inactivity. It will be closed in 2 weeks if no further activity occurs. If this issue is still relevant, please simply write any comment. Even if closed, you can still revive the issue at any time or discuss it on the dev@druid.apache.org list. Thank you for your contributions.

  3. stale commented on Apr 9, 2020

    @stale

    This issue has been marked as stale due to 280 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If this issue is still relevant, please simply write any comment. Even if closed, you can still revive the issue at any time or discuss it on the dev@druid.apache.org list. Thank you for your contributions.

  4. stale commented on May 7, 2020

    @stale

    This issue has been closed due to lack of activity. If you think that is incorrect, or the issue requires additional review, you can revive the issue at any time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions