Re: [PR] [INLONG-10866][SDK] Transform SQL support sign function [inlong]

2024-08-27 Thread via GitHub
aloyszhang merged PR #10924: URL: https://github.com/apache/inlong/pull/10924 -- 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: commits-unsubscr...@inlong

[PR] [INLONG-10866][SDK] Transform SQL support sign function [inlong]

2024-08-27 Thread via GitHub
Zkplo opened a new pull request, #10924: URL: https://github.com/apache/inlong/pull/10924 Fixes #10866 ### Motivation Add one arithmetic function classe: SignFunction. Also, add the corresponding unit test codes. ### Modifications ### Veri

Re: [PR] [INLONG-10866][SDK] Transform SQL support sign function [inlong]

2024-08-27 Thread via GitHub
Zkplo closed pull request #10868: [INLONG-10866][SDK] Transform SQL support sign function URL: https://github.com/apache/inlong/pull/10868 -- 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 specifi

Re: [PR] [INLONG-10866][SDK] Transform SQL support sign function [inlong]

2024-08-27 Thread via GitHub
aloyszhang commented on PR #10868: URL: https://github.com/apache/inlong/pull/10868#issuecomment-2312402333 ![image](https://github.com/user-attachments/assets/cf0e2be9-d90b-4855-b56e-1e01f4231514) I'm afraid there is something wrong when rebasing the master, please do it the right way.

Re: [PR] [INLONG-10866][SDK] Transform SQL support sign function [inlong]

2024-08-26 Thread via GitHub
aloyszhang commented on PR #10868: URL: https://github.com/apache/inlong/pull/10868#issuecomment-2311534726 plz rebase the master to resolve the conflict -- 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

Re: [PR] [INLONG-10866][SDK] Transform SQL support sign function [inlong]

2024-08-26 Thread via GitHub
Zkplo commented on code in PR #10868: URL: https://github.com/apache/inlong/pull/10868#discussion_r1730935050 ## inlong-sdk/transform-sdk/src/main/java/org/apache/inlong/sdk/transform/process/function/SignFunction.java: ## @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Software

Re: [PR] [INLONG-10866][SDK] Transform SQL support sign function [inlong]

2024-08-26 Thread via GitHub
aloyszhang commented on code in PR #10868: URL: https://github.com/apache/inlong/pull/10868#discussion_r1730865531 ## inlong-sdk/transform-sdk/src/main/java/org/apache/inlong/sdk/transform/process/function/SignFunction.java: ## @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Sof

[PR] [INLONG-10866][SDK] Transform SQL support sign function [inlong]

2024-08-23 Thread via GitHub
Zkplo opened a new pull request, #10868: URL: https://github.com/apache/inlong/pull/10868 Fixes #10866 ### Motivation Add one arithmetic function classe: SignFunction. Also, add the corresponding unit test codes. ### Modifications ###