[ 
https://issues.apache.org/jira/browse/HUDI-987?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yajun Luo updated HUDI-987:
---------------------------
    Description: 
This came up during review: [https://github.com/apache/hudi/pull/1677]

It added a processing logic for decimal type in AbstractRealtimeRecordReader. 
Now we need to make sure existing integration tests for MOR cover the case.

The below PR seems to have modified the trip schema in HoodieTestDataGenerator 
to add a decimal field, and we may use it to test MOR table as well as perform 
upserts on them and then query using hive.
 here: [https://github.com/apache/hudi/pull/1690]

  was:
This came up during review: 
https://github.com/apache/hudi/pull/1677

It added a processing logic for decimal type in AbstractRealtimeRecordReader. 
Now we need to make sure existing integration tests for MOR cover the case.

The below PR seems to have modified the trip schema in HoodieTestDataGenerator 
to add a decimal field, and we may use it to test MOR table as well as perform 
upserts on them and then query using hive.
here: https://github.com/apache/hudi/pull/1690


> Need integration tests for MOR table of decimal type
> ----------------------------------------------------
>
>                 Key: HUDI-987
>                 URL: https://issues.apache.org/jira/browse/HUDI-987
>             Project: Apache Hudi
>          Issue Type: Test
>          Components: Testing
>            Reporter: Yajun Luo
>            Priority: Major
>
> This came up during review: [https://github.com/apache/hudi/pull/1677]
> It added a processing logic for decimal type in AbstractRealtimeRecordReader. 
> Now we need to make sure existing integration tests for MOR cover the case.
> The below PR seems to have modified the trip schema in 
> HoodieTestDataGenerator to add a decimal field, and we may use it to test MOR 
> table as well as perform upserts on them and then query using hive.
>  here: [https://github.com/apache/hudi/pull/1690]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to