devmadhuu commented on code in PR #9258:
URL: https://github.com/apache/ozone/pull/9258#discussion_r2885119183
##########
hadoop-ozone/recon/src/test/java/org/apache/hadoop/ozone/recon/persistence/AbstractReconSqlDBTest.java:
##########
@@ -179,12 +179,12 @@ public String getDriverClass() {
@Override
public String getJdbcUrl() {
return "jdbc:derby:" + tempDir.getAbsolutePath() +
- File.separator + "derby_recon.db";
+ File.separator + "derby_recon.db;create=true";
}
@Override
public String getUserName() {
- return null;
+ return "RECON";
Review Comment:
Good catch, I was doing some test on performance with multiple users with
multiple DBs. Those changes were not needed for this PR. I reverted them to
keep the test DB wiring unchanged from baseline.
--
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]
