Skip to content
Snippets Groups Projects
Commit 3d3cf2dd authored by Nils Christian Ehmke's avatar Nils Christian Ehmke
Browse files

Fixed some CS and PMD issues

parent eddc809d
No related branches found
No related tags found
No related merge requests found
...@@ -42,9 +42,9 @@ public final class AggregatedOperationCall extends AbstractOperationCall<Aggrega ...@@ -42,9 +42,9 @@ public final class AggregatedOperationCall extends AbstractOperationCall<Aggrega
this.setStackSize(call.getStackSize()); this.setStackSize(call.getStackSize());
} }
public AggregatedOperationCall(final String container, final String component, final String operation, final String failedCause, final long totalDuration, public AggregatedOperationCall(final String container, final String component, final String operation, // NOPMD (a long parameter list cannot be avoided)
final long medianDuration, final String failedCause, final long totalDuration, final long medianDuration, final long minDuration, final long maxDuration, final long meanDuration,
final long minDuration, final long maxDuration, final long meanDuration, final int calls) { final int calls) {
super(container, component, operation); super(container, component, operation);
this.totalDuration = totalDuration; this.totalDuration = totalDuration;
......
/***************************************************************************
* Copyright 2014 Kieker Project (http://kieker-monitoring.net)
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
***************************************************************************/
package kieker.diagnosis.common.model.importer.stages; package kieker.diagnosis.common.model.importer.stages;
import kieker.diagnosis.common.domain.OperationCall; import kieker.diagnosis.common.domain.OperationCall;
......
...@@ -43,8 +43,8 @@ public final class OperationCallHandlerComposite extends CompositeStage { ...@@ -43,8 +43,8 @@ public final class OperationCallHandlerComposite extends CompositeStage {
private final CollectorSink<AggregatedOperationCall> aggCallCollector; private final CollectorSink<AggregatedOperationCall> aggCallCollector;
private final CollectorSink<AggregatedOperationCall> aggFailedCallCollector; private final CollectorSink<AggregatedOperationCall> aggFailedCallCollector;
public OperationCallHandlerComposite(final List<OperationCall> operationCalls, final List<OperationCall> failedOperationCalls, final List<AggregatedOperationCall> aggOperationCalls, public OperationCallHandlerComposite(final List<OperationCall> operationCalls, final List<OperationCall> failedOperationCalls,
final List<AggregatedOperationCall> aggFailedOperationCalls) { final List<AggregatedOperationCall> aggOperationCalls, final List<AggregatedOperationCall> aggFailedOperationCalls) {
this.operationCallExtractor = new OperationCallExtractor(); this.operationCallExtractor = new OperationCallExtractor();
this.callCollector = new CollectorSink<>(operationCalls); this.callCollector = new CollectorSink<>(operationCalls);
final Distributor<OperationCall> distributor1 = new Distributor<>(new CopyByReferenceStrategy()); final Distributor<OperationCall> distributor1 = new Distributor<>(new CopyByReferenceStrategy());
......
...@@ -40,7 +40,7 @@ public final class Mapper<I, O> { ...@@ -40,7 +40,7 @@ public final class Mapper<I, O> {
return null; return null;
} }
public class To { public final class To {
private final I key; private final I key;
...@@ -48,7 +48,7 @@ public final class Mapper<I, O> { ...@@ -48,7 +48,7 @@ public final class Mapper<I, O> {
this.key = key; this.key = key;
} }
public void to(final O value) { public void to(final O value) { // NOPMD (the method name may be short, but this is acceptable in this case)
Mapper.this.internalMap.put(this.key, value); Mapper.this.internalMap.put(this.key, value);
} }
......
...@@ -5,7 +5,7 @@ ...@@ -5,7 +5,7 @@
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
* You may obtain a copy of the License at * You may obtain a copy of the License at
* *
* http://www.apache.org/licenses/LICENSE-2.0 * http://www.apache.org/licenses/LICENSE-2.0
* *
* Unless required by applicable law or agreed to in writing, software * Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS, * distributed under the License is distributed on an "AS IS" BASIS,
......
...@@ -5,7 +5,7 @@ ...@@ -5,7 +5,7 @@
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
* You may obtain a copy of the License at * You may obtain a copy of the License at
* *
* http://www.apache.org/licenses/LICENSE-2.0 * http://www.apache.org/licenses/LICENSE-2.0
* *
* Unless required by applicable law or agreed to in writing, software * Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS, * distributed under the License is distributed on an "AS IS" BASIS,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment