You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Ao Yuchen (Jira)" <ji...@apache.org> on 2024/01/05 06:25:00 UTC

[jira] [Commented] (FLINK-33938) Correct implicit coercions in relational operators to adopt typescript 5.0

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

Ao Yuchen commented on FLINK-33938:
-----------------------------------

Hi [~Wencong Liu] , I had created a PR to solve this issue, would you like to take a look?

> Correct implicit coercions in relational operators to adopt typescript 5.0
> --------------------------------------------------------------------------
>
>                 Key: FLINK-33938
>                 URL: https://issues.apache.org/jira/browse/FLINK-33938
>             Project: Flink
>          Issue Type: Improvement
>          Components: Runtime / Web Frontend
>    Affects Versions: 1.19.0
>            Reporter: Ao Yuchen
>            Assignee: Ao Yuchen
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.19.0, 1.17.3, 1.18.2
>
>
> Since TypeScript 5.0, there is a break change that implicit coercions in relational operators are forbidden [1].
> So that the following code in flink-runtime-web/web-dashboard/src/app/components/humanize-date.pipe.ts get error:
> {code:java}
> public transform(
>   value: number | string | Date,
>   ...
> ): string | null | undefined {
>   if (value == null || value === '' || value !== value || value < 0) {
>     return '-';
>   } 
>   ...
> }{code}
> The correctness improvement is availble in here [2][.|https://github.com/microsoft/TypeScript/pull/52048.]
> I think we should optimize this type of code for better compatibility.
>  
> [1] [https://devblogs.microsoft.com/typescript/announcing-typescript-5-0/#forbidden-implicit-coercions-in-relational-operators]
> [2] [https://github.com/microsoft/TypeScript/pull/52048|https://github.com/microsoft/TypeScript/pull/52048.]



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