kurtostfeld commented on PR #25896:
URL: https://github.com/apache/flink/pull/25896#issuecomment-2572500238

   The following test failure is caused by a test file, external to the main 
git repo, that I don't have access to change. It simply needs Class<Byte> 
changed to Class<? extends Byte>.
   
   ```
   /tmp/junit13815350649750541837/CustomKryo.java:6: error: CustomKryo is not 
abstract and does not override abstract method read(Kryo,Input,Class<? extends 
Byte>) in Serializer
   public class CustomKryo extends Serializer<Byte> implements Serializable {
   ```
   
   This error with table seems like a flaky test that is unrelated to the 
changes in this PR:
   
   ```
   Jan 05 06:56:38 06:56:38.023 [ERROR]   
AdaptiveJoinTest.testWithShuffleHashJoin:47 ast ==> expected: <
   Jan 05 06:56:38 LogicalProject(a1=[$0], b1=[$1], c1=[$2], d1=[$3], a2=[$4], 
b2=[$5], c2=[$6], d2=[$7])
   Jan 05 06:56:38 +- LogicalFilter(condition=[=($0, $4)])
   Jan 05 06:56:38    +- LogicalJoin(condition=[true], joinType=[inner])
   Jan 05 06:56:38       :- LogicalTableScan(table=[[default_catalog, 
default_database, T1, source: [TestTableSource(a1, b1, c1, d1)]]])
   Jan 05 06:56:38       +- LogicalTableScan(table=[[default_catalog, 
default_database, T2, source: [TestTableSource(a2, b2, c2, d2)]]])
   Jan 05 06:56:38 > but was: <
   Jan 05 06:56:38 LogicalProject(a1=[$0], b1=[$1], c1=[$2], d1=[$3], a2=[$4], 
b2=[$5], c2=[$6], d2=[$7])
   Jan 05 06:56:38 +- LogicalFilter(condition=[=($0, $4)])
   Jan 05 06:56:38    +- LogicalJoin(condition=[true], joinType=[inner])
   Jan 05 06:56:38       :- LogicalTableScan(table=[[default_catalog, 
default_database, T1]])
   Jan 05 06:56:38       +- LogicalTableScan(table=[[default_catalog, 
default_database, T2]])
   ```


-- 
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: issues-unsubscr...@flink.apache.org

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

Reply via email to