Skip to content

Conversation

@lxfeng1997
Copy link
Contributor

What this PR does:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


AsterZephyr and others added 6 commits June 8, 2025 21:13
…nsert on duplicate apache#704 (apache#725)

* bugfix apache#704

* bugfix apache#704

* bugfix704-2

* bugfix-test-2

* bugfix-test-2

* pr725 bugfix

---------

Co-authored-by: JayLiu <[email protected]>
Co-authored-by: FengZhang <[email protected]>
…nadh (apache#772)

* fix: Solve the conflict problem of introducing multiple versions of knadh

* fix: fix ci fail

---------

Co-authored-by: JayLiu <[email protected]>
Co-authored-by: FengZhang <[email protected]>
…uest. (apache#764)

* feat: add unit test workflow

* feat:the ability to automatically run unit tests after creating a pull request.

* feat:the ability to automatically run unit tests after creating a pull request.

* feat:the ability to automatically run unit tests after creating a pull request.

* feat:the ability to automatically run unit tests after creating a pull request.

* feat:the ability to automatically run unit tests after creating a pull request.

* feat:the ability to automatically run unit tests after creating a pull request.

* Optimize/at build lock key performance  (apache#837)

* Refer to buildlockkey2 optimization apache#829

* Time complexity O(NM)-> O(NK) about buildlockkey and buildlockkey2  Increased readability  apache#829

* update import sort apache#829

* update Encapsulation into util packages apache#829

* Support Update join (apache#761)

* duplicate image row for update join

* update join condition placeholder param error

* update join bugfix

* Open test annotations

* recover update executor

* recover update test

* recover update test

* modified version param

---------

Co-authored-by: JayLiu <[email protected]>
Co-authored-by: FengZhang <[email protected]>

---------

Co-authored-by: jimin <[email protected]>
Co-authored-by: JayLiu <[email protected]>
Co-authored-by: FengZhang <[email protected]>
Co-authored-by: Wiggins <[email protected]>
Co-authored-by: lxfeng1997 <[email protected]>
# Conflicts:
#	.github/workflows/build.yml
#	go.mod
#	pkg/datasource/sql/exec/at/base_executor_test.go
#	pkg/datasource/sql/util/lockkey.go
@github-actions github-actions bot added bug Something isn't working ci/cd module/at coding and removed bug Something isn't working module/at labels Sep 14, 2025
@lxfeng1997 lxfeng1997 added bug Something isn't working module/at labels Sep 14, 2025
@codecov-commenter
Copy link

codecov-commenter commented Sep 14, 2025

Codecov Report

❌ Patch coverage is 75.20661% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.45%. Comparing base (4d2e765) to head (f86d5c2).

Files with missing lines Patch % Lines
...mysql_insertonduplicate_update_undo_log_builder.go 74.35% 23 Missing and 7 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #884      +/-   ##
==========================================
+ Coverage   53.37%   53.45%   +0.08%     
==========================================
  Files         208      208              
  Lines       12659    12718      +59     
==========================================
+ Hits         6757     6799      +42     
- Misses       5401     5418      +17     
  Partials      501      501              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lxfeng1997 lxfeng1997 self-assigned this Sep 19, 2025
Copy link
Contributor

@Code-Fight Code-Fight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Code-Fight Code-Fight merged commit c0a7ab0 into apache:master Nov 8, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci/cd coding module/at

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants