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

Implement large array constructing through chunk concatenation #22622

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Implement large array constructing through chunk concatenation #22622

wants to merge 8 commits into from

Commits on Apr 28, 2024

  1. Enable array with sizes of 255 or more to be created 

    - Updated SqlToRowExpressionTranslator.java
- Split array into chunks of 200 and concatenated if the size is greater than 200
    @arararz
    arararz committed Apr 28, 2024
    Configuration menu
    Copy the full SHA
    6a09f4b View commit details
    Browse the repository at this point in the history

  2. Update TestArrayFunctions.java 

    - Added test case to ensure that creating arrays of size 255 or more are now supported
    @arararz
    arararz committed Apr 28, 2024
    Configuration menu
    Copy the full SHA
    914f26b View commit details
    Browse the repository at this point in the history

  3. Integrated Test Cases for Large Array Construction 

    Tested for arrays with size larger than 200
Checked that each chunk/sublist is concatenated correctly
    @parthr01
    parthr01 committed Apr 28, 2024
    Configuration menu
    Copy the full SHA
    c9a3b77 View commit details
    Browse the repository at this point in the history

  4. Revert "Integrated Test Cases for Large Array Construction" 

    This reverts commit c9a3b77.
    @arararz
    arararz committed Apr 28, 2024
    Configuration menu
    Copy the full SHA
    b289112 View commit details
    Browse the repository at this point in the history

  5. Integrated Test Cases for LargeArray Construction (formatting) 

    Attempted to fix formatting Issues
    @parthr01
    parthr01 committed Apr 28, 2024
    Configuration menu
    Copy the full SHA
    d403789 View commit details
    Browse the repository at this point in the history

  6. Revert "Integrated Test Cases for LargeArray Construction (formatting)" 

    This reverts commit d403789.
    @arararz
    arararz committed Apr 28, 2024
    Configuration menu
    Copy the full SHA
    ed56d08 View commit details
    Browse the repository at this point in the history

  7. Integrated Test Cases for Large Array Construction 

    - Tested for arrays with size larger than 200
- Checked that each chunk/sublist is concatenated correctly
    @arararz
    arararz committed Apr 28, 2024
    Configuration menu
    Copy the full SHA
    a23e778 View commit details
    Browse the repository at this point in the history

  8. Style changes 

    - Fix style issues in SqlToRowExpressionTranslator.java and TestRowExpressionSerde.java
    @arararz
    arararz committed Apr 28, 2024
    Configuration menu
    Copy the full SHA
    a3bf120 View commit details
    Browse the repository at this point in the history