cadl opened a new issue #2063:
URL: https://github.com/apache/hudi/issues/2063


   **_Tips before filing an issue_**
   
   - Have you gone through our 
[FAQs](https://cwiki.apache.org/confluence/display/HUDI/FAQ)?
   
   - Join the mailing list to engage in conversations and get faster support at 
[email protected].
   
   - If you have triaged this as a bug, then file an 
[issue](https://issues.apache.org/jira/projects/HUDI/issues) directly.
   
   **Describe the problem you faced**
   
   I have an existed COW table foo (int a, string b). When I set 
`hoodie.avro.schema.validate` to `true`, upsert a dataframe(long a, string b) 
on table foo,  "schema compatibility check failed" will throw.
   
   **To Reproduce**
   
   Steps to reproduce the behavior:
   
   1. write existed data, with column type int
   
   ``` scala
   val existedDf = spark.sql("select 1 as a, '' as b, 1 as __row_key, 0 as 
__row_version")
   val testTable = "foo"
   
   
   val testConf = Map(
         "hoodie.table.name" -> testTable,
         "hoodie.avro.schema.validate" -> "true",
         "hoodie.datasource.write.recordkey.field" -> "__row_key",
         "hoodie.datasource.write.table.name" -> testTable,
         "hoodie.datasource.write.precombine.field" -> "__row_version",
         "hoodie.datasource.write.partitionpath.field" -> "",
         "hoodie.datasource.write.keygenerator.class" -> 
classOf[org.apache.hudi.keygen.NonpartitionedKeyGenerator].getName,
         "hoodie.datasource.write.hive_style_partitioning" -> "true",
         "hoodie.datasource.write.operation" -> "upsert"        
       )
   
   
existedDf.write.format("org.apache.hudi").options(testConf).mode("append").save(s"file:///jfs/cadl/hudi_data/schema/foo")
   ```
   
   2. write new data, with column type long
   
   ``` scala
   val newDf = spark.sql("select cast(1 as long) as a, '' as b, 1 as __row_key, 
1 as __row_version")
   
newDf.write.format("org.apache.hudi").options(testConf).mode("append").save(s"file:///jfs/cadl/hudi_data/schema/foo")
   ```
   
   Then the exception raised. If I remove `hoodie.avro.schema.validate` from 
`testConf`, upsert successfully.
   
   
   **Expected behavior**
   
   Upsert and read successfully.
   
   **Environment Description**
   
   * Hudi version : 0.6.0
   
   * Spark version : 2.4.3
   
   * Hive version :
   
   * Hadoop version :
   
   * Storage (HDFS/S3/GCS..) : LocalFile
   
   * Running on Docker? (yes/no) :
   
   
   **Additional context**
   
   Are the params of TableSchemaResolver.isSchemaCompatible is inverted? 
   
   
https://github.com/apache/hudi/blob/6461927eac3f8a225b17af5ecb6ace8c9cf1757b/hudi-client/src/main/java/org/apache/hudi/table/HoodieTable.java#L557
   
   **Stacktrace**
   
   ```
   org.apache.hudi.exception.HoodieUpsertException: Failed upsert schema 
compatibility check.
     at 
org.apache.hudi.table.HoodieTable.validateUpsertSchema(HoodieTable.java:572)
     at 
org.apache.hudi.client.HoodieWriteClient.upsert(HoodieWriteClient.java:190)
     at 
org.apache.hudi.DataSourceUtils.doWriteOperation(DataSourceUtils.java:260)
     at 
org.apache.hudi.HoodieSparkSqlWriter$.write(HoodieSparkSqlWriter.scala:169)
     at org.apache.hudi.DefaultSource.createRelation(DefaultSource.scala:125)
     at 
org.apache.spark.sql.execution.datasources.SaveIntoDataSourceCommand.run(SaveIntoDataSourceCommand.scala:45)
     at 
org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult$lzycompute(commands.scala:70)
     at 
org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult(commands.scala:68)
     at 
org.apache.spark.sql.execution.command.ExecutedCommandExec.doExecute(commands.scala:86)
     at 
org.apache.spark.sql.execution.SparkPlan$$anonfun$execute$1.apply(SparkPlan.scala:131)
     at 
org.apache.spark.sql.execution.SparkPlan$$anonfun$execute$1.apply(SparkPlan.scala:127)
     at 
org.apache.spark.sql.execution.SparkPlan$$anonfun$executeQuery$1.apply(SparkPlan.scala:155)
     at 
org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:151)
     at 
org.apache.spark.sql.execution.SparkPlan.executeQuery(SparkPlan.scala:152)
     at org.apache.spark.sql.execution.SparkPlan.execute(SparkPlan.scala:127)
     at 
org.apache.spark.sql.execution.QueryExecution.toRdd$lzycompute(QueryExecution.scala:80)
     at 
org.apache.spark.sql.execution.QueryExecution.toRdd(QueryExecution.scala:80)
     at 
org.apache.spark.sql.DataFrameWriter$$anonfun$runCommand$1.apply(DataFrameWriter.scala:676)
     at 
org.apache.spark.sql.DataFrameWriter$$anonfun$runCommand$1.apply(DataFrameWriter.scala:676)
     at 
org.apache.spark.sql.execution.SQLExecution$$anonfun$withNewExecutionId$1.apply(SQLExecution.scala:78)
     at 
org.apache.spark.sql.execution.SQLExecution$.withSQLConfPropagated(SQLExecution.scala:125)
     at 
org.apache.spark.sql.execution.SQLExecution$.withNewExecutionId(SQLExecution.scala:73)
     at 
org.apache.spark.sql.DataFrameWriter.runCommand(DataFrameWriter.scala:676)
     at 
org.apache.spark.sql.DataFrameWriter.saveToV1Source(DataFrameWriter.scala:285)
     at org.apache.spark.sql.DataFrameWriter.save(DataFrameWriter.scala:271)
     at org.apache.spark.sql.DataFrameWriter.save(DataFrameWriter.scala:229)
     ... 69 elided
   Caused by: org.apache.hudi.exception.HoodieException: Failed schema 
compatibility check for writerSchema 
:{"type":"record","name":"foo_record","namespace":"hoodie.foo","fields":[{"name":"_hoodie_commit_time","type":["null","string"],"doc":"","default":null},{"name":"_hoodie_commit_seqno","type":["null","string"],"doc":"","default":null},{"name":"_hoodie_record_key","type":["null","string"],"doc":"","default":null},{"name":"_hoodie_partition_path","type":["null","string"],"doc":"","default":null},{"name":"_hoodie_file_name","type":["null","string"],"doc":"","default":null},{"name":"a","type":"long"},{"name":"b","type":"string"},{"name":"__row_key","type":"int"},{"name":"__row_version","type":"int"}]},
 table schema 
:{"type":"record","name":"foo_record","namespace":"hoodie.foo","fields":[{"name":"_hoodie_commit_time","type":["null","string"],"doc":"","default":null},{"name":"_hoodie_commit_seqno","type":["null","string"],"doc":"","default":null},{"name":"_hoodie_record_key","type":["
 
null","string"],"doc":"","default":null},{"name":"_hoodie_partition_path","type":["null","string"],"doc":"","default":null},{"name":"_hoodie_file_name","type":["null","string"],"doc":"","default":null},{"name":"a","type":"int"},{"name":"b","type":"string"},{"name":"__row_key","type":"int"},{"name":"__row_version","type":"int"}]},
 base path :file:///jfs/cadl/hudi_data/schema/foo
     at org.apache.hudi.table.HoodieTable.validateSchema(HoodieTable.java:564)
     at 
org.apache.hudi.table.HoodieTable.validateUpsertSchema(HoodieTable.java:570)
     ... 94 more
   ```
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to