Github user twalthr commented on the issue:
https://github.com/apache/flink/pull/3426
Thanks for the update @haohui. I fixed the decimal issues by applying
changes from @wuchong's PR. I think for now we should not do the primitive
checking for scalar functions, this should be part of
Github user haohui commented on the issue:
https://github.com/apache/flink/pull/3426
@twalthr can you please take another look? Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fe
Github user haohui commented on the issue:
https://github.com/apache/flink/pull/3426
Fix the unit tests.
There are two additional changes:
1. There are precision differences when converting `double` to
`BigDecimal`. Fix the unit tests.
2. When registering UDFs Fli
Github user haohui commented on the issue:
https://github.com/apache/flink/pull/3426
Sorry stuck with something. Will update the PR in a day or two.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user twalthr commented on the issue:
https://github.com/apache/flink/pull/3426
@haohui your build is failing. Can you have a look at it again?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does no
Github user twalthr commented on the issue:
https://github.com/apache/flink/pull/3426
Thanks @wuchong and @haohui. I will look at it today.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have thi
Github user wuchong commented on the issue:
https://github.com/apache/flink/pull/3426
Hi @haohui , the code looks very good to me. Thanks for investigating this,
waiting for the CI pass.
@twalthr , I think you would like to have a look.
---
If your project is set up for it,