wuchong commented on issue #9193: [FLINK-13318][table-planner-blink] Fix Blink 
planner tests failing on Scala 2.12 by setting SerialVersionUID of UDF 
explicitly
URL: https://github.com/apache/flink/pull/9193#issuecomment-514028089
 
 
   Hi @twalthr , this is because 
   
   1. we print the udf's md5 into the node explanation.
   2. blink planner uses xml to verify plan, the xml content is immutable 
between different Scala versions. 
   3. the serialization behavior is different between Scala 2.11 and 2.12 when 
version id is not defined. 
   
   That's why we add version id to all existing scala udfs. But this doesn't 
mean we force users to set this. This is just a workaround for the plan test 
for different Scala versions. Users can still use a Scala udf without version 
id in blink planner and flink planner. 
   
   > why are only Blink tests affected by this?
   
   Flink uses `"${func.functionIdentifier}"` to generate plan content, so that 
it can adapt to different Scala versions automatically.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to