You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@seatunnel.apache.org by "JohnZp (via GitHub)" <gi...@apache.org> on 2023/09/28 06:17:12 UTC

[GitHub] [seatunnel] JohnZp opened a new pull request, #5579: [Fix]: fix the cdc bug about NPE when the original table deletes a field

JohnZp opened a new pull request, #5579:
URL: https://github.com/apache/seatunnel/pull/5579

   
   ### Purpose of this pull request
   
   Fix NPE, it introduced by https://github.com/apache/seatunnel/issues/5575
   
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   
   ### How was this patch tested?
   Nothing
   
   
   ### Check list
   
   * [ ] If any new Jar binary package adding in your PR, please add License Notice according
     [New License Guide](https://github.com/apache/seatunnel/blob/dev/docs/en/contribution/new-license.md)
   * [ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
   * [ ] If you are contributing the connector code, please check that the following files are updated:
     1. Update change log that in connector document. For more details you can refer to [connector-v2](https://github.com/apache/seatunnel/tree/dev/docs/en/connector-v2)
     2. Update [plugin-mapping.properties](https://github.com/apache/seatunnel/blob/dev/plugin-mapping.properties) and add new connector information in it
     3. Update the pom file of [seatunnel-dist](https://github.com/apache/seatunnel/blob/dev/seatunnel-dist/pom.xml)
   * [ ] Update the [`release-note`](https://github.com/apache/seatunnel/blob/dev/release-note.md).


-- 
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] [seatunnel] JohnZp commented on pull request #5579: [Fix] Fix the cdc bug about NPE when the original table deletes a field

Posted by "JohnZp (via GitHub)" <gi...@apache.org>.
JohnZp commented on PR #5579:
URL: https://github.com/apache/seatunnel/pull/5579#issuecomment-1738658492

   @Hisoka-X The checked have passed, please help review


-- 
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] [seatunnel] Hisoka-X commented on pull request #5579: [Fix] Fix the cdc bug about NPE when the original table deletes a field

Posted by "Hisoka-X (via GitHub)" <gi...@apache.org>.
Hisoka-X commented on PR #5579:
URL: https://github.com/apache/seatunnel/pull/5579#issuecomment-1738551781

   Could you open your fork repository action follow the guide?
   ![image](https://github.com/apache/seatunnel/assets/32387433/bfbe20ff-855c-4cdc-96fd-6ffb8e3e9f1c)
   


-- 
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


Re: [PR] [Fix] Fix the cdc bug about NPE when the original table deletes a field [seatunnel]

Posted by "liugddx (via GitHub)" <gi...@apache.org>.
liugddx merged PR #5579:
URL: https://github.com/apache/seatunnel/pull/5579


-- 
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