Skip to content
This repository was archived by the owner on Sep 27, 2019. It is now read-only.

Commit 642d027

Browse files
committed
Additional fixes
1. Add Makefile, override .gitignore for junit tests (for now) 2. Fix independent bug of segfault when invoking -display_settings
1 parent 7d8d579 commit 642d027

File tree

3 files changed

+39
-1
lines changed

3 files changed

+39
-1
lines changed

script/testing/junit/.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
2+
!Makefile

script/testing/junit/Makefile

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
2+
3+
4+
.SUFFIXES: .java .class
5+
6+
CP = .:lib/junit-4.12.jar:lib/hamcrest-core-1.3.jar:lib/postgresql-9.4.1209.jre6.jar
7+
JC = javac
8+
JFLAGS = -g
9+
JUNIT = org.junit.runner.JUnitCore
10+
RM = rm
11+
12+
.java.class:
13+
$(JC) $(JFLAGS) -cp $(CP) $*.java
14+
15+
CLASSES = \
16+
PLTestBase.java \
17+
InsertTPCCTest.java \
18+
InsertPSTest.java \
19+
UpdateTest.java
20+
21+
default: classes
22+
23+
classes: $(CLASSES:.java=.class)
24+
25+
clean:
26+
$(RM) *.class
27+
28+
test: $(CLASSES)
29+
java -jar lib/junit-platform-console-standalone-1.1.0.jar \
30+
--class-path $(CP) \
31+
-c InsertPSTest
32+
# -c UpdateTest
33+
# -c InsertPSTest -c InsertTPCCTest
34+
35+
36+

src/settings/settings_manager.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ const std::string SettingsManager::GetInfo() const {
114114
info.append(
115115
StringUtil::Format("%28s: %-28s\n", "Code-generation",
116116
GetBool(SettingId::codegen) ? "enabled" : "disabled"));
117-
info.append(StringUtil::Format("%28s: %-28s\n", "Optimization Timeout",
117+
info.append(StringUtil::Format("%28s: %-28i\n", "Optimization Timeout",
118118
GetInt(SettingId::task_execution_timeout)));
119119
return StringBoxUtil::Box(info);
120120
}

0 commit comments

Comments
 (0)