Timm0 commented on PR #28979: URL: https://github.com/apache/flink/pull/28979#issuecomment-6218212568
> Logically I agree with you, but as I noted [here](https://github.com/apache/flink/pull/28979#discussion_r3856334151) `IS_NOT_NULL` was already a thing before this change. From a user perspective I'd expect `NaN` operations to mirror `Null` operations so if we remove the negative for `Nan` I think we should do the same for `Null` which would be a breaking change. > > The argument could be made that `IS_NOT_NULL` is tech debt that shouldn't be brought forward to `NaN`, but if we went that route I 'd expect a warning or deprecation notice be tied to the `Null` operator. > > I'm obviously new to this project so feel free to correct my analysis if I'm missing historical context or something. Thanks @MattBelle for the explanation. As I said I'm not against having `isNotNan`. My main "concern" here is that `IS NOT NULL` is standard SQL syntax, while for `IS_NAN` and `IS_NOT_NAN` other engines like Spark/BigQuery/Trino also only have `IS_NAN`. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
