Skip to content

11743: remove placeholder OBJECT column data type.#11816

Closed
abhioncbr wants to merge 1 commit intoapache:masterfrom
abhioncbr:11743-remove-object=data-schema
Closed

11743: remove placeholder OBJECT column data type.#11816
abhioncbr wants to merge 1 commit intoapache:masterfrom
abhioncbr:11743-remove-object=data-schema

Conversation

@abhioncbr
Copy link
Contributor

As per the issue, deleting the placeholder OBJECT data type.

@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2023

Codecov Report

Merging #11816 (589fda1) into master (a5b3483) will decrease coverage by 48.75%.
Report is 3 commits behind head on master.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #11816       +/-   ##
=============================================
- Coverage     63.14%   14.40%   -48.75%     
+ Complexity     1140      201      -939     
=============================================
  Files          2343     2343               
  Lines        126420   126428        +8     
  Branches      19442    19442               
=============================================
- Hits          79832    18212    -61620     
- Misses        40914   106669    +65755     
+ Partials       5674     1547     -4127     
Flag Coverage Δ
custom-integration1 <0.01% <ø> (ø)
integration <0.01% <ø> (ø)
integration1 <0.01% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 <0.01% <ø> (-63.13%) ⬇️
java-17 14.40% <ø> (-48.63%) ⬇️
java-20 14.39% <ø> (+<0.01%) ⬆️
temurin 14.40% <ø> (-48.75%) ⬇️
unittests 14.40% <ø> (-48.75%) ⬇️
unittests1 ?
unittests2 14.40% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...rg/apache/pinot/common/function/FunctionUtils.java 0.00% <ø> (-96.88%) ⬇️

... and 1511 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

put(float[].class, ColumnDataType.FLOAT_ARRAY);
put(double[].class, ColumnDataType.DOUBLE_ARRAY);
put(String[].class, ColumnDataType.STRING_ARRAY);
put(Object.class, ColumnDataType.OBJECT);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing this leads it to return null; hence, we set the _resultMetadata in ScalarTransformationFunction or in PostAggregationFunction as String. Eventually, it fails while evaluating the valueblock.

@Jackie-Jiang / @walterddr do you guys think determining the _resultMetadata based on the transformation function result makes sense like we are doing in CaseTransformFunction

@abhioncbr abhioncbr closed this Mar 7, 2024
@abhioncbr abhioncbr deleted the 11743-remove-object=data-schema branch April 25, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants