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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
