You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by "dockerzhang (via GitHub)" <gi...@apache.org> on 2023/03/01 03:04:48 UTC
[GitHub] [inlong] dockerzhang opened a new pull request, #7471: [INLONG-7299][Sort] Optimize the options check for the Kafka connector
dockerzhang opened a new pull request, #7471:
URL: https://github.com/apache/inlong/pull/7471
### Prepare a Pull Request
*(Change the title refer to the following example)*
- Title Example: [INLONG-XYZ][Component] Title of the pull request
*(The following *XYZ* should be replaced by the actual [GitHub Issue](https://github.com/apache/inlong/issues) number)*
- Fixes #7299
### Motivation
*Explain here the context, and why you're making that change. What is the problem you're trying to solve?*
### Modifications
- All supported formats are checked in `ExtractNodeUtils.parsingFormat()`, so it does not need to check again for Kafka ExtractNode.
- Add `valueFormatPrefix` check for `inlong-msg.' prefix options.
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] featzhang commented on pull request #7471: [INLONG-7299][Sort] Optimize the options check for the Kafka connector
Posted by "featzhang (via GitHub)" <gi...@apache.org>.
featzhang commented on PR #7471:
URL: https://github.com/apache/inlong/pull/7471#issuecomment-1449271798
+1
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] dockerzhang merged pull request #7471: [INLONG-7299][Sort] Optimize the options check for the Kafka connector
Posted by "dockerzhang (via GitHub)" <gi...@apache.org>.
dockerzhang merged PR #7471:
URL: https://github.com/apache/inlong/pull/7471
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] featzhang commented on pull request #7471: [INLONG-7299][Sort] Optimize the options check for the Kafka connector
Posted by "featzhang (via GitHub)" <gi...@apache.org>.
featzhang commented on PR #7471:
URL: https://github.com/apache/inlong/pull/7471#issuecomment-1449266333
LGTM
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org