[ 
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)

Reply via email to