{"push_id":1405872608,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"7569cf6cb85bda7d0e76d3e75e286d4796e77e08","before":"a75e3fe923372c56bc1b2f4baeaaf5868ad28341","commits":[{"sha":"7569cf6cb85bda7d0e76d3e75e286d4796e77e08","author":{"name":"Xianyang Liu","email":"aa7698c29891c5a517fa7b8cd0b48e8488977230@icloud.com"},"message":"[SPARK-18420][BUILD] Fix the errors caused by lint check in Java\n\n## What changes were proposed in this pull request?\n\nSmall fix, fix the errors caused by lint check in Java\n\n- Clear unused objects and `UnusedImports`.\n- Add comments around the method `finalize` of `NioBufferedFileInputStream`to turn off checkstyle.\n- Cut the line which is longer than 100 characters into two lines.\n\n## How was this patch tested?\nTravis CI.\n```\n$ build/mvn -T 4 -q -DskipTests -Pyarn -Phadoop-2.3 -Pkinesis-asl -Phive -Phive-thriftserver install\n$ dev/lint-java\n```\nBefore:\n```\nCheckstyle checks failed at following occurrences:\n[ERROR] src/main/java/org/apache/spark/network/util/TransportConf.java:[21,8] (imports) UnusedImports: Unused import - org.apache.commons.crypto.cipher.CryptoCipherFactory.\n[ERROR] src/test/java/org/apache/spark/network/sasl/SparkSaslSuite.java:[516,5] (modifier) RedundantModifier: Redundant 'public' modifier.\n[ERROR] src/main/java/org/apache/spark/io/NioBufferedFileInputStream.java:[133] (coding) NoFinalizer: Avoid using finalizer method.\n[ERROR] src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeMapData.java:[71] (sizes) LineLength: Line is longer than 100 characters (found 113).\n[ERROR] src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java:[112] (sizes) LineLength: Line is longer than 100 characters (found 110).\n[ERROR] src/test/java/org/apache/spark/sql/catalyst/expressions/HiveHasherSuite.java:[31,17] (modifier) ModifierOrder: 'static' modifier out of order with the JLS suggestions.\n[ERROR]src/main/java/org/apache/spark/examples/ml/JavaLogisticRegressionWithElasticNetExample.java:[64] (sizes) LineLength: Line is longer than 100 characters (found 103).\n[ERROR] src/main/java/org/apache/spark/examples/ml/JavaInteractionExample.java:[22,8] (imports) UnusedImports: Unused import - org.apache.spark.ml.linalg.Vectors.\n[ERROR] src/main/java/org/apache/spark/examples/ml/JavaInteractionExample.java:[51] (regexp) RegexpSingleline: No trailing whitespace allowed.\n```\n\nAfter:\n```\n$ build/mvn -T 4 -q -DskipTests -Pyarn -Phadoop-2.3 -Pkinesis-asl -Phive -Phive-thriftserver install\n$ dev/lint-java\nUsing `mvn` from path: /home/travis/build/ConeyLiu/spark/build/apache-maven-3.3.9/bin/mvn\nCheckstyle checks passed.\n```\n\nAuthor: Xianyang Liu <xyliu0530@icloud.com>\n\nCloses #15865 from ConeyLiu/master.","distinct":true,"url":"https://api.github.com/repos/dbtsai/spark/commits/7569cf6cb85bda7d0e76d3e75e286d4796e77e08"}]}
{"actor":{"display_login":"dbtsai-bot"}}