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)