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

[DO NOT MERGE - triggering tests] Egalpin/refactor broker request handle request #14213

Draft
wants to merge 48 commits into
base: master
Choose a base branch
from
Draft

[DO NOT MERGE - triggering tests] Egalpin/refactor broker request handle request #14213

wants to merge 48 commits into from

Commits on Aug 3, 2024

  1. Allows multiple requests per server per request ID

    @egalpin
    egalpin committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    2b339af View commit details
    Browse the repository at this point in the history

  2. Spotless linter

    @egalpin
    egalpin committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    4e8656e View commit details
    Browse the repository at this point in the history

  3. Fixes test setups broken by method signature changes

    @egalpin
    egalpin committed Aug 3, 2024
    Configuration menu
    Copy the full SHA
    73748fd View commit details
    Browse the repository at this point in the history

Commits on Aug 6, 2024

  1. Moves placement of appending tableName metadata

    @egalpin
    egalpin committed Aug 6, 2024
    Configuration menu
    Copy the full SHA
    c8285ec View commit details
    Browse the repository at this point in the history

  2. Removes unused import

    @egalpin
    egalpin committed Aug 6, 2024
    Configuration menu
    Copy the full SHA
    35188ea View commit details
    Browse the repository at this point in the history

  3. Updates signatures to match

    @egalpin
    egalpin committed Aug 6, 2024
    Configuration menu
    Copy the full SHA
    2ef9047 View commit details
    Browse the repository at this point in the history

  4. Makes QueryResponse return iterable of ServerResponses without nested… 

    … Map
    @egalpin
    egalpin committed Aug 6, 2024
    Configuration menu
    Copy the full SHA
    a9eaa1f View commit details
    Browse the repository at this point in the history

  5. Updates scala to match iterable server responses

    @egalpin
    egalpin committed Aug 6, 2024
    Configuration menu
    Copy the full SHA
    09d9e20 View commit details
    Browse the repository at this point in the history

Commits on Aug 7, 2024

  1. Updates license header to comply with linter

    @egalpin
    egalpin committed Aug 7, 2024
    Configuration menu
    Copy the full SHA
    a692f62 View commit details
    Browse the repository at this point in the history

  2. Makes query routing table non-nullable

    @egalpin
    egalpin committed Aug 7, 2024
    Configuration menu
    Copy the full SHA
    5359950 View commit details
    Browse the repository at this point in the history

  3. Moves table name metadata setting to server-side

    @egalpin
    egalpin committed Aug 7, 2024
    Configuration menu
    Copy the full SHA
    f441389 View commit details
    Browse the repository at this point in the history

  4. Duplicate addition of table name?

    @egalpin
    egalpin committed Aug 7, 2024
    Configuration menu
    Copy the full SHA
    7cac310 View commit details
    Browse the repository at this point in the history

Commits on Aug 8, 2024

  1. Ensures async query response map employs ConcurrentHashMap at the nes… 

    …ted level
    @egalpin
    egalpin committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    c41813a View commit details
    Browse the repository at this point in the history

  2. WIP - fixing tests where tablename with type is now required

    @egalpin
    egalpin committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    df17781 View commit details
    Browse the repository at this point in the history

  3. Removing unnecessary table name metadata set ops

    @egalpin
    egalpin committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    9088b46 View commit details
    Browse the repository at this point in the history

  4. Updates more tests to be compatible

    @egalpin
    egalpin committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    3124c99 View commit details
    Browse the repository at this point in the history

  5. Ensures query_hash is set everywhere query_id is

    @egalpin
    egalpin committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    31a19c7 View commit details
    Browse the repository at this point in the history

  6. Linting fixes

    @egalpin
    egalpin committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    343104a View commit details
    Browse the repository at this point in the history

  7. Merge remote-tracking branch 'upstream/master' into egalpin/refactor-… 

    …broker-request
    @egalpin
    egalpin committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    9fb4537 View commit details
    Browse the repository at this point in the history

Commits on Aug 9, 2024

  1. Fixes bug in broker reduce service, remove empty serverInstance entries

    @egalpin
    egalpin committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    6248be0 View commit details
    Browse the repository at this point in the history

  2. Updates data table reducers to check size number of data tables as op… 

    …posed to map entries
    @egalpin
    egalpin committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    30933ac View commit details
    Browse the repository at this point in the history

  3. Ensures no concurrent modification of nested map

    @egalpin
    egalpin committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    ff55b30 View commit details
    Browse the repository at this point in the history

  4. Adds comment RE deployment

    @egalpin
    egalpin committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    8c8c006 View commit details
    Browse the repository at this point in the history

Commits on Aug 12, 2024

  1. Uses count of issued queries rather than distinct server count for se… 

    …tting QueryOptionKey.SERVER_RETURN_FINAL_RESULT
    @egalpin
    egalpin committed Aug 12, 2024
    Configuration menu
    Copy the full SHA
    7eac151 View commit details
    Browse the repository at this point in the history

Commits on Aug 13, 2024

  1. Ensures table name is always added to DataTable metadata

    @egalpin
    egalpin committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    c3d9bf8 View commit details
    Browse the repository at this point in the history

  2. Fixes use of wrong hashcode

    @egalpin
    egalpin committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    75db44f View commit details
    Browse the repository at this point in the history

  3. Removes codeblock handled elsewhere

    @egalpin
    egalpin committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    2ffd65d View commit details
    Browse the repository at this point in the history

Commits on Sep 3, 2024

  1. WIP - use TableName instead of query hash

    @egalpin
    egalpin committed Sep 3, 2024
    Configuration menu
    Copy the full SHA
    efc72fb View commit details
    Browse the repository at this point in the history

Commits on Oct 1, 2024

  1. Borrows 1 digit from request-id

    @egalpin
    egalpin committed Oct 1, 2024
    Configuration menu
    Copy the full SHA
    859311d View commit details
    Browse the repository at this point in the history

Commits on Oct 8, 2024

  1. WIP - Use tableName not query hash

    @egalpin
    egalpin committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    5ed97ac View commit details
    Browse the repository at this point in the history

  2. Use requestId to infer tableType if null

    @egalpin
    egalpin committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    4941c0b View commit details
    Browse the repository at this point in the history

  3. Merge remote-tracking branch 'upstream/master' into egalpin/refactor-… 

    …broker-request
    @egalpin
    egalpin committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    64305d5 View commit details
    Browse the repository at this point in the history

  4. Fixes post-merge import

    @egalpin
    egalpin committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    0cd47ca View commit details
    Browse the repository at this point in the history

  5. Adds shared constants for request ID hacking

    @egalpin
    egalpin committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    c280cb8 View commit details
    Browse the repository at this point in the history

  6. Linting fixes

    @egalpin
    egalpin committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    c5837bd View commit details
    Browse the repository at this point in the history

Commits on Oct 9, 2024

  1. Adds hacky access to table type inference from request ID

    @egalpin
    egalpin committed Oct 9, 2024
    Configuration menu
    Copy the full SHA
    598553e View commit details
    Browse the repository at this point in the history

Commits on Oct 10, 2024

  1. Removes QUERY_HASH from DataTable metadata

    @egalpin
    egalpin committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    f7fc1fe View commit details
    Browse the repository at this point in the history

  2. Renames Constants to BrokerRequestIdConstants for clarity

    @egalpin
    egalpin committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    7fb325a View commit details
    Browse the repository at this point in the history

  3. Moves inferTableType to DataTableUtils

    @egalpin
    egalpin committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    996a6f9 View commit details
    Browse the repository at this point in the history

  4. Clean up references to tableName

    @egalpin
    egalpin committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    eb10a8b View commit details
    Browse the repository at this point in the history

  5. Updates PrioritySchedulerTest#testSubmitBeforeRunning to expect that … 

    …server always sends tableName in meta
    @egalpin
    egalpin committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    a9c8d2e View commit details
    Browse the repository at this point in the history

  6. Linting fixes

    @egalpin
    egalpin committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    734189b View commit details
    Browse the repository at this point in the history

Commits on Oct 11, 2024

  1. Empty commit retrigger tests

    @egalpin
    egalpin committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    4d621fa View commit details
    Browse the repository at this point in the history

Commits on Oct 17, 2024

  1. Makes requestId manipulations idempotent

    @egalpin
    egalpin committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    41aa2e1 View commit details
    Browse the repository at this point in the history

  2. Adds license to BrokerRequestIdUtilsTest.java

    @egalpin
    egalpin committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    edc5aa4 View commit details
    Browse the repository at this point in the history

  3. Fixes spotless violations

    @egalpin
    egalpin committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    86c1d4c View commit details
    Browse the repository at this point in the history

  4. WIP - make handleRequest own less stuff

    @egalpin
    egalpin committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    c7f56cc View commit details
    Browse the repository at this point in the history

  5. Adds CompiledPinotQuery inner class

    @egalpin
    egalpin committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    0b53a3d View commit details
    Browse the repository at this point in the history