You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@seatunnel.apache.org by GitBox <gi...@apache.org> on 2022/06/05 15:34:32 UTC

[GitHub] [incubator-seatunnel] CalvinKirs opened a new pull request, #1985: [Improve]Use Jackson replace Fastjson (#1971)

CalvinKirs opened a new pull request, #1985:
URL: https://github.com/apache/incubator-seatunnel/pull/1985

   
   ## Check list
   
   * [ ] Code changed are covered with tests, or it does not need tests for reason:
   * [ ] If any new Jar binary package adding in your PR, please add License Notice according
     [New License Guide](https://github.com/apache/incubator-seatunnel/blob/dev/docs/en/contribution/new-license.md)
   * [ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
   


-- 
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-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-seatunnel] CalvinKirs commented on pull request #1985: [Improve]Use Jackson replace Fastjson (#1971)

Posted by GitBox <gi...@apache.org>.
CalvinKirs commented on PR #1985:
URL: https://github.com/apache/incubator-seatunnel/pull/1985#issuecomment-1146995293

   > Hi Kris, #1971 will release on 2.1.2, Do we must merge this PR into 2.1.2 to reduce risk?
   
    I think it's better best


-- 
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-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-seatunnel] BenJFan commented on pull request #1985: [Improve]Use Jackson replace Fastjson (#1971)

Posted by GitBox <gi...@apache.org>.
BenJFan commented on PR #1985:
URL: https://github.com/apache/incubator-seatunnel/pull/1985#issuecomment-1146978323

   Hi Kris, #1971 will release on 2.1.2, Do we must merge this PR into 2.1.2 to reduce risk?


-- 
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-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-seatunnel] CalvinKirs merged pull request #1985: [Improve]Use Jackson replace Fastjson (#1971)

Posted by GitBox <gi...@apache.org>.
CalvinKirs merged PR #1985:
URL: https://github.com/apache/incubator-seatunnel/pull/1985


-- 
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-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org