You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "github-actions[bot] (via GitHub)" <gi...@apache.org> on 2026/05/06 16:16:24 UTC

[PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

github-actions[bot] opened a new pull request, #63038:
URL: https://github.com/apache/doris/pull/63038

   Cherry-picked from #62141


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4504264277

   @iaorekhov-1980  So I think you should not depend on auto pick pull request. You need pick the PR manually and submit a PR to branch 4.1. And if this is OK, I will close this PR.


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei closed pull request #63038: branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141
URL: https://github.com/apache/doris/pull/63038


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "iaorekhov-1980 (via GitHub)" <gi...@apache.org>.
iaorekhov-1980 commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4451715379

   run build compile


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4394298962

   run buildall


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "iaorekhov-1980 (via GitHub)" <gi...@apache.org>.
iaorekhov-1980 commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4451720646

   run feut


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "iaorekhov-1980 (via GitHub)" <gi...@apache.org>.
iaorekhov-1980 commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4395459025

   hello, @yiguolei 
   Could you please provide output of failed FE_UT test?
   Is it something serious or just flaky test?
   Because original PR has successfully passed FE_UT in https://github.com/apache/doris/pull/62141


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4497316424

   run buildall


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4414374583

   @iaorekhov-1980 iaorekhov-1980


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "iaorekhov-1980 (via GitHub)" <gi...@apache.org>.
iaorekhov-1980 commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4486882723

   Hello, @yiguolei 
   Could you please re-execute the merge as Copilot completely ignored this PR: [refactor](fe) Replace all JMockit usage with Mockito and remove JMockit dependency - #62221 
   So all imports related to Mockito framework have been missed in cherry pick merge


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4389997268

   
   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR).
   
   Please clearly describe your PR:
   1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
   2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
   3. What features were added. Why was this function added?
   4. Which code was refactored and why was this part of the code refactored?
   5. Which functions were optimized and what is the difference before and after the optimization?
       


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "iaorekhov-1980 (via GitHub)" <gi...@apache.org>.
iaorekhov-1980 commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4498165943

   Hello, @yiguolei 
   The same problem during compilation of feut has occurred :(
   Because copilot originally missed mandatory import statements in EncryptSQLTest.java in the initial cherry picking commit [0dea8bf.](https://github.com/apache/doris/commit/0dea8bf337023b7fc579f32f52b8de762489a329#diff-3859cc7ecae2735eb318bb385281df595262df9a90f165639ce5b704cc368e10R407-R428)


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4414374233

   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66233_66233&logFilter=debug&logView=flowAware)  [INFO] Total time:  09:39 min
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66234_66234&logFilter=debug&logView=flowAware)  [INFO] Finished at: 2026-05-10T10:31:58+08:00
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66235_66235&logFilter=debug&logView=flowAware)  [INFO] ------------------------------------------------------------------------
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66236_66236&logFilter=debug&logView=flowAware)  [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.10.1:testCompile (default-testCompile) on project fe-core: Compilation failure: Compilation failure:
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66237_66237&logFilter=debug&logView=flowAware)  [ERROR] /root/doris/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java:[411,14] cannot find symbol
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66238_66238&logFilter=debug&logView=flowAware)  [ERROR]   symbol:   class MockedConstruction
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66239_66239&logFilter=debug&logView=flowAware)  [ERROR]   location: class org.apache.doris.nereids.parser.EncryptSQLTest
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66240_66240&logFilter=debug&logView=flowAware)  [ERROR] /root/doris/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java:[411,57] cannot find symbol
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66241_66241&logFilter=debug&logView=flowAware)  [ERROR]   symbol:   variable Mockito
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66242_66242&logFilter=debug&logView=flowAware)  [ERROR]   location: class org.apache.doris.nereids.parser.EncryptSQLTest
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66243_66243&logFilter=debug&logView=flowAware)  [ERROR] /root/doris/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java:[412,54] cannot find symbol
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66244_66244&logFilter=debug&logView=flowAware)  [ERROR]   symbol:   variable Mockito
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66245_66245&logFilter=debug&logView=flowAware)  [ERROR]   location: class org.apache.doris.nereids.parser.EncryptSQLTest
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66246_66246&logFilter=debug&logView=flowAware)  [ERROR] /root/doris/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java:[412,17] cannot find symbol
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66247_66247&logFilter=debug&logView=flowAware)  [ERROR]   symbol:   variable Mockito
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66248_66248&logFilter=debug&logView=flowAware)  [ERROR]   location: class org.apache.doris.nereids.parser.EncryptSQLTest
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66249_66249&logFilter=debug&logView=flowAware)  [ERROR] /root/doris/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java:[414,21] cannot find symbol
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66250_66250&logFilter=debug&logView=flowAware)  [ERROR]   symbol:   variable Mockito
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66251_66251&logFilter=debug&logView=flowAware)  [ERROR]   location: class org.apache.doris.nereids.parser.EncryptSQLTest
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66252_66252&logFilter=debug&logView=flowAware)  [ERROR] /root/doris/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java:[415,21] cannot find symbol
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66253_66253&logFilter=debug&logView=flowAware)  [ERROR]   symbol:   class Profile
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66254_66254&logFilter=debug&logView=flowAware)  [ERROR]   location: class org.apache.doris.nereids.parser.EncryptSQLTest
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66255_66255&logFilter=debug&logView=flowAware)  [ERROR] /root/doris/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java:[415,43] cannot find symbol
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66256_66256&logFilter=debug&logView=flowAware)  [ERROR]   symbol:   class Profile
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66257_66257&logFilter=debug&logView=flowAware)  [ERROR]   location: class org.apache.doris.nereids.parser.EncryptSQLTest
   [10:31:58 ](http://43.132.222.7:8111/buildConfiguration/Doris_Doris_FeUt/941527?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandPull+Request+Details=true&expandBuildChangesSection=true&expandBuildProblemsSection=true&showLog=941527_66258_66258&logFilter=debug&logView=flowAware)  [ERROR] /root/doris/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java:[417,21] cannot find symbol


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4414206443

   run buildall


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] branch-4.1: [enhance](nereids) improve masking of user's password for ALTER USER and CREATE USER commands in audit logs #62141 [doris]

Posted by "iaorekhov-1980 (via GitHub)" <gi...@apache.org>.
iaorekhov-1980 commented on PR #63038:
URL: https://github.com/apache/doris/pull/63038#issuecomment-4428674141

   hello, @yiguolei 
   I see an issue in automated merge performed by copilot for EncryptSQLTest.java
   https://github.com/apache/doris/blob/0dea8bf337023b7fc579f32f52b8de762489a329/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java#L18
   
   Please pay attention - that it has missed all imports which are currently available in master branch, like this one
   https://github.com/apache/doris/blob/058d97897f611b0bf68965d073a5888aafba9805/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java#L38
   
   These changes were not introduced by me, but included earlier by this commit https://github.com/apache/doris/pull/62221 [[refactor] (fe) Replace all JMockit usage with Mockito and remove JMockit dependency]
   
   https://github.com/apache/doris/blob/833a8a8f98b7ca41fd07857d273dbff9432b4705/fe/fe-core/src/test/java/org/apache/doris/nereids/parser/EncryptSQLTest.java#L39
   
   Could you please re-execute the merge and include mentioned imports in the EncryptSQLTest.java?
   The same issue is relevant for https://github.com/apache/doris/pull/63037


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org