[
https://issues.apache.org/jira/browse/BEAM-933?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Daniel Halperin updated BEAM-933:
---------------------------------
Description:
{code}
[INFO] --- findbugs-maven-plugin:3.0.1:check (default) @ beam-examples-java ---
[INFO] BugInstance size is 4
[INFO] Error size is 0
[INFO] Total bugs: 4
[INFO] Possible null pointer dereference in
org.apache.beam.examples.complete.TfIdf.listInputDocuments(TfIdf$Options) due
to return value of called method [org.apache.beam.examples.complete.TfIdf,
org.apache.beam.examples.complete.TfIdf] Dereferenced at TfIdf.java:[line
124]Known null at TfIdf.java:[line 124]
[INFO] Bad attempt to compute absolute value of signed 32-bit hashcode in
org.apache.beam.examples.complete.TopWikipediaSessions$ComputeTopSessions$1.processElement(DoFn$ProcessContext)
[org.apache.beam.examples.complete.TopWikipediaSessions$ComputeTopSessions$1]
At TopWikipediaSessions.java:[line 165]
[INFO] org.apache.beam.examples.complete.TrafficRoutes$StationSpeed defines
compareTo(TrafficRoutes$StationSpeed) and uses Object.equals()
[org.apache.beam.examples.complete.TrafficRoutes$StationSpeed] At
TrafficRoutes.java:[line 113]
[INFO]
org.apache.beam.examples.cookbook.TriggerExample$InsertDelays.processElement(DoFn$ProcessContext)
uses the nextDouble method of Random to generate a random integer; using
nextInt is more efficient
[org.apache.beam.examples.cookbook.TriggerExample$InsertDelays] At
TriggerExample.java:[line 479]
[INFO]
{code}
was:
{code}
[INFO] --- findbugs-maven-plugin:3.0.1:check (default) @ beam-runners-spark ---
[INFO] BugInstance size is 19
[INFO] Error size is 0
[INFO] Total bugs: 19
[INFO] instanceof will always return false in
org.apache.beam.runners.spark.SparkRunner.run(Pipeline), since a
RuntimeException can't be a org.apache.spark.SparkException
[org.apache.beam.runners.spark.SparkRunner] At SparkRunner.java:[line 161]
[INFO]
org.apache.beam.runners.spark.aggregators.metrics.WithNamedAggregatorsSupport$1.apply(Map$Entry)
makes inefficient use of keySet iterator instead of entrySet iterator
[org.apache.beam.runners.spark.aggregators.metrics.WithNamedAggregatorsSupport$1]
At WithNamedAggregatorsSupport.java:[line 125]
[INFO] The class name
org.apache.beam.runners.spark.aggregators.metrics.sink.CsvSink shadows the
simple name of the superclass org.apache.spark.metrics.sink.CsvSink
[org.apache.beam.runners.spark.aggregators.metrics.sink.CsvSink] At
CsvSink.java:[lines 37-38]
[INFO] The class name
org.apache.beam.runners.spark.aggregators.metrics.sink.GraphiteSink shadows the
simple name of the superclass org.apache.spark.metrics.sink.GraphiteSink
[org.apache.beam.runners.spark.aggregators.metrics.sink.GraphiteSink] At
GraphiteSink.java:[lines 37-38]
[INFO] t must be non-null but is marked as nullable
[org.apache.beam.runners.spark.translation.WindowingHelpers$4] At
WindowingHelpers.java:[line 88]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$1.evaluate(ConsoleIO$Write$Unbound,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$1]
At StreamingTransformTranslator.java:[line 92]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$10.evaluate(ParDo$Bound,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$10]
At StreamingTransformTranslator.java:[line 360]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$11.evaluate(ParDo$BoundMulti,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$11]
At StreamingTransformTranslator.java:[line 395]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$2.evaluate(Read$Unbounded,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$2]
At StreamingTransformTranslator.java:[line 104]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$3.evaluate(CreateStream$QueuedValues,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$3]
At StreamingTransformTranslator.java:[line 115]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$4.evaluate(Flatten$FlattenPCollectionList,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$4]
At StreamingTransformTranslator.java:[line 128]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$5.evaluate(Window$Bound,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$5]
At StreamingTransformTranslator.java:[line 167]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$6.evaluate(GroupByKey,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$6]
At StreamingTransformTranslator.java:[line 215]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$7.evaluate(Combine$GroupedValues,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$7]
At StreamingTransformTranslator.java:[line 251]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$8.evaluate(Combine$Globally,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$8]
At StreamingTransformTranslator.java:[line 281]
[INFO] Unchecked/unconfirmed cast from
org.apache.beam.runners.spark.translation.EvaluationContext to
org.apache.beam.runners.spark.translation.streaming.StreamingEvaluationContext
in
org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$9.evaluate(Combine$PerKey,
EvaluationContext)
[org.apache.beam.runners.spark.translation.streaming.StreamingTransformTranslator$9]
At StreamingTransformTranslator.java:[line 321]
[INFO] Inconsistent synchronization of
org.apache.beam.runners.spark.util.BroadcastHelper$CodedBroadcastHelper.value;
locked 75% of time
[org.apache.beam.runners.spark.util.BroadcastHelper$CodedBroadcastHelper,
org.apache.beam.runners.spark.util.BroadcastHelper$CodedBroadcastHelper,
org.apache.beam.runners.spark.util.BroadcastHelper$CodedBroadcastHelper,
org.apache.beam.runners.spark.util.BroadcastHelper$CodedBroadcastHelper]
Unsynchronized access at BroadcastHelper.java:[line 111]Synchronized access at
BroadcastHelper.java:[line 103]Synchronized access at
BroadcastHelper.java:[line 106]Synchronized access at
BroadcastHelper.java:[line 104]
[INFO] Inconsistent synchronization of
org.apache.beam.runners.spark.util.BroadcastHelper$DirectBroadcastHelper.bcast;
locked 50% of time
[org.apache.beam.runners.spark.util.BroadcastHelper$DirectBroadcastHelper,
org.apache.beam.runners.spark.util.BroadcastHelper$DirectBroadcastHelper]
Unsynchronized access at BroadcastHelper.java:[line 81]Synchronized access at
BroadcastHelper.java:[line 74]
[INFO] Inconsistent synchronization of
org.apache.beam.runners.spark.util.BroadcastHelper$DirectBroadcastHelper.value;
locked 75% of time
[org.apache.beam.runners.spark.util.BroadcastHelper$DirectBroadcastHelper,
org.apache.beam.runners.spark.util.BroadcastHelper$DirectBroadcastHelper,
org.apache.beam.runners.spark.util.BroadcastHelper$DirectBroadcastHelper,
org.apache.beam.runners.spark.util.BroadcastHelper$DirectBroadcastHelper]
Unsynchronized access at BroadcastHelper.java:[line 81]Synchronized access at
BroadcastHelper.java:[line 73]Synchronized access at BroadcastHelper.java:[line
76]Synchronized access at BroadcastHelper.java:[line 74]
[INFO]
{code}
> Findbugs doesn't pass in Java Examples
> --------------------------------------
>
> Key: BEAM-933
> URL: https://issues.apache.org/jira/browse/BEAM-933
> Project: Beam
> Issue Type: Bug
> Components: sdk-java-extensions
> Reporter: Daniel Halperin
> Assignee: Davor Bonaci
>
> {code}
> [INFO] --- findbugs-maven-plugin:3.0.1:check (default) @ beam-examples-java
> ---
> [INFO] BugInstance size is 4
> [INFO] Error size is 0
> [INFO] Total bugs: 4
> [INFO] Possible null pointer dereference in
> org.apache.beam.examples.complete.TfIdf.listInputDocuments(TfIdf$Options) due
> to return value of called method [org.apache.beam.examples.complete.TfIdf,
> org.apache.beam.examples.complete.TfIdf] Dereferenced at TfIdf.java:[line
> 124]Known null at TfIdf.java:[line 124]
> [INFO] Bad attempt to compute absolute value of signed 32-bit hashcode in
> org.apache.beam.examples.complete.TopWikipediaSessions$ComputeTopSessions$1.processElement(DoFn$ProcessContext)
>
> [org.apache.beam.examples.complete.TopWikipediaSessions$ComputeTopSessions$1]
> At TopWikipediaSessions.java:[line 165]
> [INFO] org.apache.beam.examples.complete.TrafficRoutes$StationSpeed defines
> compareTo(TrafficRoutes$StationSpeed) and uses Object.equals()
> [org.apache.beam.examples.complete.TrafficRoutes$StationSpeed] At
> TrafficRoutes.java:[line 113]
> [INFO]
> org.apache.beam.examples.cookbook.TriggerExample$InsertDelays.processElement(DoFn$ProcessContext)
> uses the nextDouble method of Random to generate a random integer; using
> nextInt is more efficient
> [org.apache.beam.examples.cookbook.TriggerExample$InsertDelays] At
> TriggerExample.java:[line 479]
> [INFO]
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)