[ 
https://issues.apache.org/jira/browse/FLINK-33727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17792538#comment-17792538
 ] 

Jim Hughes commented on FLINK-33727:
------------------------------------

{quote}if so I'm curious whether it would be more helpful for others to have at 
least a comment about that in sources
{quote}
Absolutely!  The RestoreTest framework is new, and this discussion shows that 
there are a number of non-obvious assumptions.  [~bvarghese] and I are new to 
Flink, and as we've worked with it, we have extended it to have the features 
necessary to test various capabilities.

I apologize for taking out CI!  The order merges resulted in PRs not testing 
all of the programs together in one branch until they were merged.  (Which is 
why I'm willing to suggest disabling all RestoreTests temporarily or reverting 
the commits which caused the issue.)

As an additional improvement to the RestoreTestBase, we could save the SQL text 
in a file and fail if it is changed.  Of course, then folks could still update 
the test files which ought to be "immutable" in some sense.

> JoinRestoreTest is failing on AZP
> ---------------------------------
>
>                 Key: FLINK-33727
>                 URL: https://issues.apache.org/jira/browse/FLINK-33727
>             Project: Flink
>          Issue Type: Bug
>          Components: Table SQL / Planner
>    Affects Versions: 1.19.0
>            Reporter: Sergey Nuyanzin
>            Priority: Critical
>              Labels: pull-request-available, test-stability
>
> Since {{JoinRestoreTest}} was introduced in FLINK-33470 it seems to be a 
> reason
> {noformat}
> Dec 02 04:42:26 04:42:26.408 [ERROR] Failures: 
> Dec 02 04:42:26 04:42:26.408 [ERROR]   
> JoinRestoreTest>RestoreTestBase.testRestore:283 
> Dec 02 04:42:26 Expecting actual:
> Dec 02 04:42:26   ["+I[9, carol, apple, 9000]",
> Dec 02 04:42:26     "+I[8, bill, banana, 8000]",
> Dec 02 04:42:26     "+I[6, jerry, pen, 6000]"]
> Dec 02 04:42:26 to contain exactly in any order:
> Dec 02 04:42:26   ["+I[Adam, null]",
> Dec 02 04:42:26     "+I[Baker, Research]",
> Dec 02 04:42:26     "+I[Charlie, Human Resources]",
> Dec 02 04:42:26     "+I[Charlie, HR]",
> Dec 02 04:42:26     "+I[Don, Sales]",
> Dec 02 04:42:26     "+I[Victor, null]",
> Dec 02 04:42:26     "+I[Helena, Engineering]",
> Dec 02 04:42:26     "+I[Juliet, Engineering]",
> Dec 02 04:42:26     "+I[Ivana, Research]",
> Dec 02 04:42:26     "+I[Charlie, People Operations]"]
> {noformat}
> examples of failures
> https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=55120&view=logs&j=0c940707-2659-5648-cbe6-a1ad63045f0a&t=075c2716-8010-5565-fe08-3c4bb45824a4&l=12099
> https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=55129&view=logs&j=a9db68b9-a7e0-54b6-0f98-010e0aff39e2&t=cdd32e0b-6047-565b-c58f-14054472f1be&l=11786
> https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=55136&view=logs&j=0c940707-2659-5648-cbe6-a1ad63045f0a&t=075c2716-8010-5565-fe08-3c4bb45824a4&l=12099
> https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=55137&view=logs&j=0c940707-2659-5648-cbe6-a1ad63045f0a&t=075c2716-8010-5565-fe08-3c4bb45824a4&l=11779



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to