Skip to content

Commit 5d70561

Browse files
committed
fix lint errors
1 parent 150d19c commit 5d70561

File tree

3 files changed

+8
-5
lines changed

3 files changed

+8
-5
lines changed

dev/archery/archery/benchmark/jmh.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@ def __init__(self, build, benchmark_filter=None):
5454
...
5555
[INFO]
5656
"""
57+
5758
def list_benchmarks(self):
5859
argv = []
5960
if self.benchmark_filter:
@@ -110,7 +111,8 @@ def __init__(self, benchmark, primaryMetric,
110111
"measurementTime": counters["measurementTime"],
111112
"jvmArgs": counters["jvmArgs"]
112113
}
113-
self.reciprocal_value = True if self.score_unit.endswith("/op") else False
114+
self.reciprocal_value = True if self.score_unit.endswith(
115+
"/op") else False
114116
if self.score_unit.startswith("ops/"):
115117
idx = self.score_unit.find("/")
116118
self.normalizePerSec(self.score_unit[idx+1:])

dev/archery/archery/cli.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -435,7 +435,7 @@ def benchmark_list(ctx, rev_or_path, src, preserve, output, cmake_extras,
435435
src, root, rev_or_path, conf)
436436

437437
elif language == "java":
438-
for key in { 'cpp_package_prefix', 'cxx_flags', 'cxx', 'cc'}:
438+
for key in {'cpp_package_prefix', 'cxx_flags', 'cxx', 'cc'}:
439439
del kwargs[key]
440440
conf = JavaBenchmarkRunner.default_configuration(
441441
java_home=java_home, java_options=java_options,
@@ -509,7 +509,7 @@ def benchmark_run(ctx, rev_or_path, src, preserve, output, cmake_extras,
509509
suite_filter=suite_filter, benchmark_filter=benchmark_filter)
510510

511511
elif language == "java":
512-
for key in { 'cpp_package_prefix', 'cxx_flags', 'cxx', 'cc'}:
512+
for key in {'cpp_package_prefix', 'cxx_flags', 'cxx', 'cc'}:
513513
del kwargs[key]
514514
conf = JavaBenchmarkRunner.default_configuration(
515515
java_home=java_home, java_options=java_options,
@@ -637,7 +637,7 @@ def benchmark_diff(ctx, src, preserve, output, language, cmake_extras,
637637
benchmark_filter=benchmark_filter)
638638

639639
elif language == "java":
640-
for key in { 'cpp_package_prefix', 'cxx_flags', 'cxx', 'cc'}:
640+
for key in {'cpp_package_prefix', 'cxx_flags', 'cxx', 'cc'}:
641641
del kwargs[key]
642642
conf = JavaBenchmarkRunner.default_configuration(
643643
java_home=java_home, java_options=java_options,

dev/archery/archery/lang/java.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,8 @@ def __init__(self,
4444
self.java_options = java_options
4545

4646
self.build_extras = list(build_extras) if build_extras else []
47-
self.benchmark_extras = list(benchmark_extras) if benchmark_extras else []
47+
self.benchmark_extras = list(
48+
benchmark_extras) if benchmark_extras else []
4849

4950
@property
5051
def build_definitions(self):

0 commit comments

Comments
 (0)