You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "lvyanquan (via GitHub)" <gi...@apache.org> on 2023/03/31 05:05:58 UTC

[GitHub] [hudi] lvyanquan opened a new pull request, #8334: [MINOR][DOCS] Remove preCombineField which is not in table

lvyanquan opened a new pull request, #8334:
URL: https://github.com/apache/hudi/pull/8334

   ### Change Logs
   
   `ts` is not a column of `hudi_table_p0`, and don't need to set preCombineField for cow table.
   
   error message:
   ```
   java.lang.IllegalArgumentException: Can't find preCombineKey `ts` in root
    |-- _hoodie_commit_time: string (nullable = true)
    |-- _hoodie_commit_seqno: string (nullable = true)
    |-- _hoodie_record_key: string (nullable = true)
    |-- _hoodie_partition_path: string (nullable = true)
    |-- _hoodie_file_name: string (nullable = true)
    |-- id: long (nullable = true)
    |-- name: string (nullable = true)
    |-- dt: string (nullable = true)
    |-- hh: string (nullable = true)
   .
           at org.apache.hudi.common.util.ValidationUtils.checkArgument(ValidationUtils.java:42)
           at org.apache.spark.sql.hudi.HoodieOptionConfig$.validateTable(HoodieOptionConfig.scala:209)
           at org.apache.spark.sql.catalyst.catalog.HoodieCatalogTable.parseSchemaAndConfigs(HoodieCatalogTable.scala:260)
           at org.apache.spark.sql.catalyst.catalog.HoodieCatalogTable.initHoodieTable(HoodieCatalogTable.scala:173)
           at org.apache.spark.sql.hudi.command.CreateHoodieTableCommand.run(CreateHoodieTableCommand.scala:71)
   
   ```  
   
   ### Impact
   
   None
   
   ### Risk level (write none, low medium or high below)
   
   None
   
   ### Documentation Update
   
   [table_management.md](https://hudi.apache.org/docs/table_management#partitioned-table)
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Change Logs and Impact were stated clearly
   - [ ] Adequate tests were added if applicable
   - [ ] CI passed
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [hudi] yihua merged pull request #8334: [MINOR][DOCS] Remove preCombineField which is not in table

Posted by "yihua (via GitHub)" <gi...@apache.org>.
yihua merged PR #8334:
URL: https://github.com/apache/hudi/pull/8334


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org