reswqa
reswqa
> Why, then, does my PR appear to break this? when().then().otherwise(pl.all()).name.with_suffix() worked before, now it does not. The traversal order is `then->otherwise->when`. We have `then (literal)`, before your change, it...
But I think your changes is correct. `suffix` should take into account both `Literal` and `Alias` IMO.
> An alternative thought would be to add a suffix parameter to keep_name, which may reduce complexity with regards to the positioning of keep_name. Well, this would really make things...
Close this in favor of #9971. Feel free to reopen this.
@GOODBOY008 This change makes JDK21 unhappy. see [FLINK-35837](https://issues.apache.org/jira/browse/FLINK-35837). Would you mind taking a look? cc @1996fanrui @Jiabao-Sun
> If Flink side wants to handle this type of exception, it is best to tolerate the scenario of Celeborn upgrades, eg retry client within a reasonable time at least....
I'd like to file a PR to implement this feature.
cc @mtfelisb
@flinkbot run azure