kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a...@apache.org
Subject [kudu] 02/03: mini_hms: move derby.log into test data directory
Date Tue, 26 Mar 2019 18:40:01 GMT
This is an automated email from the ASF dual-hosted git repository.

adar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit f1533afeec1029649b7176c67f03b4210265d7dc
Author: Adar Dembo <adar@cloudera.com>
AuthorDate: Tue Mar 26 00:25:31 2019 -0700

    mini_hms: move derby.log into test data directory
    
    This is especially annoying when running Java tests, as you'll wind up with
    derby.log files in your source tree (rather than build/latest or whatever).
    
    Change-Id: I6e4750e28d20895962e28295d55ddf486f82e95d
    Reviewed-on: http://gerrit.cloudera.org:8080/12855
    Tested-by: Adar Dembo <adar@cloudera.com>
    Reviewed-by: Grant Henke <granthenke@apache.org>
    Reviewed-by: Hao Hao <hao.hao@cloudera.com>
---
 src/kudu/hms/mini_hms.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/kudu/hms/mini_hms.cc b/src/kudu/hms/mini_hms.cc
index 8cbb6de..e730f80 100644
--- a/src/kudu/hms/mini_hms.cc
+++ b/src/kudu/hms/mini_hms.cc
@@ -130,6 +130,10 @@ Status MiniHms::Start() {
     // timeout.
     "-Dderby.locks.deadlockTimeout=1";
 
+  // By default, Derby will create its log file in the current working
+  // directory. We want it in data_root_.
+  java_options += Substitute(" -Dderby.stream.error.file=$0/derby.log", data_root_);
+
   if (!krb5_conf_.empty()) {
     java_options += Substitute(" -Djava.security.krb5.conf=$0", krb5_conf_);
   }


Mime
View raw message