[chunjun][elasticsearch7]fix npe errors when column doesn't specified… by danqiusheng · Pull Request #1224 · DTStack/chunjun
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 1.7k
Merged
FlechazoW merged 1 commit intoDTStack:masterfrom
Sep 6, 2022Merged
[chunjun][elasticsearch7]fix npe errors when column doesn't specified…#1224
FlechazoW merged 1 commit intoDTStack:masterfrom
[chunjun][elasticsearch7]fix npe errors when column doesn't specified…#1224
FlechazoW merged 1 commit intoDTStack:masterfrom
Conversation
Copy link
Contributor
danqiusheng
commented
Sep 6, 2022
danqiusheng
commented
fix npe errors when column doesn't specified description.
Purpose of this pull request
Which issue you fix
internal issues.
Checklist:
- I have executed the 'mvn spotless:apply' command to format my code.
- I have a meaningful commit message (including the issue id, the template of commit message is '[label-type-#issue-id][fixed-module] a meaningful commit message.')
- I have performed a self-review of my own code.
- I have commented my code, particularly in hard-to-understand areas.
- I have made corresponding changes to the documentation.
- I have added tests that prove my fix is effective or that my feature works.
- New and existing unit tests pass locally with my changes.
- I have checked my code and corrected any misspellings.
- My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)
HiLany
requested a review
from Paddy0523
FlechazoW
merged commit
20ba28a
into
DTStack:master
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment