-
Notifications
You must be signed in to change notification settings - Fork 8.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature: support for undo full data columns on update operate #2509
Conversation
Signed-off-by: slievrly <slievrly@163.com>
for (String updateColumn : updateColumns) { | ||
selectSQLJoin.add(updateColumn); | ||
for (String column : tableMeta.getAllColumns().keySet()) { | ||
selectSQLJoin.add(column); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Columns from tableMeta, need to deal with keyword
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great found, fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
solve update times cloumn
Signed-off-by: slievrly <slievrly@163.com>
Codecov Report
@@ Coverage Diff @@
## develop #2509 +/- ##
=============================================
- Coverage 51.24% 51.20% -0.04%
Complexity 2819 2819
=============================================
Files 556 556
Lines 17807 17829 +22
Branches 2102 2109 +7
=============================================
+ Hits 9125 9130 +5
- Misses 7820 7833 +13
- Partials 862 866 +4
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Whether to add the enable the function flag? |
Signed-off-by: slievrly <slievrly@163.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: slievrly slievrly@163.com
Ⅰ. Describe what this PR did
bugfix: fix update only care about update columns
Ⅱ. Does this pull request fix one issue?
fix #2502 #2541
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews