-
Notifications
You must be signed in to change notification settings - Fork 12.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
Update MapperProxy.java #10487
Update MapperProxy.java #10487
Conversation
fix:代理类打印日志错误
plugin/datasource/src/main/java/com/alibaba/nacos/plugin/datasource/proxy/MapperProxy.java
Outdated
Show resolved
Hide resolved
fix:代理类打印日志错误
It seems ci can't pass, please fix them. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #10487 +/- ##
=============================================
- Coverage 53.30% 53.29% -0.01%
- Complexity 5446 5447 +1
=============================================
Files 895 895
Lines 28436 28440 +4
Branches 3119 3120 +1
=============================================
Hits 15157 15157
- Misses 11945 11950 +5
+ Partials 1334 1333 -1 see 3 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
@@ -35,20 +36,20 @@ | |||
* @author hyx | |||
**/ | |||
public class MapperProxy implements InvocationHandler { | |||
|
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.
The indent changed, please use nacos code style to reformat code.
reformat code
reformat code
Sorry, due to the first submission, I did not carefully read the nacos pr specification, which wasted the review time |
No problem, PR keeping enhance and fix will be merged. |
* Update MapperProxy.java fix:代理类打印日志错误 * Update MapperProxy.java fix:代理类打印日志错误 * 规范格式 * Update MapperProxy.java reformat code * Update MapperProxy.java reformat code
fix:代理类打印日志错误
#10486