Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/16500 )
Change subject: IMPALA-10198 (part 1): Unify Java in a single java/ directory ...................................................................... Patch Set 5: (87 comments) http://gerrit.cloudera.org:8080/#/c/16500/5/java/TableFlattener/src/main/java/org/apache/impala/infra/tableflattener/FileMigrator.java File java/TableFlattener/src/main/java/org/apache/impala/infra/tableflattener/FileMigrator.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/TableFlattener/src/main/java/org/apache/impala/infra/tableflattener/FileMigrator.java@82 PS5, Line 82: Record dstRecord, FlattenedSchema dstDataset, String fieldNamePrefix) { line too long (97 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java File java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@30 PS5, Line 30: // Converts CSV files for the AllTypes table into HBase shell commands that insert the same data. line too long (97 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@31 PS5, Line 31: // Reads all CSV files from the given input directory and write a single file containing all HBase commands. line too long (108 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@34 PS5, Line 34: public static void convertAllTypesTable(String inDir, String outDir, String hbaseTableName) { line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@55 PS5, Line 55: } line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@67 PS5, Line 67: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@68 PS5, Line 68: public static void convertLine(String csvLine, PrintWriter out, String hbaseTableName) throws IOException { line too long (109 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@72 PS5, Line 72: } line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@73 PS5, Line 73: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@74 PS5, Line 74: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:bigint_col", parts[5]); line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@75 PS5, Line 75: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:bool_col", parts[1]); line too long (97 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@76 PS5, Line 76: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:date_string_col", parts[8]); line too long (104 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@77 PS5, Line 77: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:double_col", parts[7]); line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@78 PS5, Line 78: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:float_col", parts[6]); line too long (98 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@79 PS5, Line 79: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:int_col", parts[4]); line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@80 PS5, Line 80: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:smallint_col", parts[3]); line too long (101 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@81 PS5, Line 81: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:string_col", parts[9]); line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@82 PS5, Line 82: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:timestamp_col", parts[10]); line too long (103 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@83 PS5, Line 83: out.format("put '%s', '%s', '%s', '%s'\n", hbaseTableName, parts[0], "d:tinyint_col", parts[2]); line too long (100 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@85 PS5, Line 85: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@86 PS5, Line 86: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@88 PS5, Line 88: * Convert CSV test data in a file with HBase shell commands for inserting the same data. line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@88 PS5, Line 88: * Convert CSV test data in a file with HBase shell commands for inserting the same data. line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@89 PS5, Line 89: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@96 PS5, Line 96: convertAllTypesTable("AllTypesError", "HBaseAllTypesError", "functional_hbase.hbasealltypeserror"); line too long (103 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@97 PS5, Line 97: convertAllTypesTable("AllTypesErrorNoNulls", "HBaseAllTypesErrorNoNulls", line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/TestDataGenerator.java File java/datagenerator/src/main/java/org/apache/impala/datagenerator/TestDataGenerator.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/datagenerator/src/main/java/org/apache/impala/datagenerator/TestDataGenerator.java@46 PS5, Line 46: GregorianCalendar endDate = new GregorianCalendar(DEFAULT_END_YEAR, Calendar.DECEMBER, 31); line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java File java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@57 PS5, Line 57: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@86 PS5, Line 86: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@99 PS5, Line 99: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@173 PS5, Line 173: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@180 PS5, Line 180: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java File java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@61 PS5, Line 61: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@64 PS5, Line 64: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@80 PS5, Line 80: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@81 PS5, Line 81: // Path to XML file containing allocations. line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@83 PS5, Line 83: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@85 PS5, Line 85: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@88 PS5, Line 88: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@91 PS5, Line 91: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@95 PS5, Line 95: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@99 PS5, Line 99: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@101 PS5, Line 101: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@145 PS5, Line 145: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@153 PS5, Line 153: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@167 PS5, Line 167: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@192 PS5, Line 192: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@196 PS5, Line 196: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@397 PS5, Line 397: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@501 PS5, Line 501: } else if ("queue".endsWith(field.getTagName()) || line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@532 PS5, Line 532: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FSQueueType.java File java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FSQueueType.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FSQueueType.java@24 PS5, Line 24: * Represents a leaf queue line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FSQueueType.java@26 PS5, Line 26: LEAF, line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FSQueueType.java@27 PS5, Line 27: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FairSchedulerConfiguration.java File java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FairSchedulerConfiguration.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FairSchedulerConfiguration.java@37 PS5, Line 37: protected static final String ALLOW_UNDECLARED_POOLS = CONF_PREFIX + "allow-undeclared-pools"; line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FairSchedulerConfiguration.java@44 PS5, Line 44: protected static final String USER_AS_DEFAULT_QUEUE = CONF_PREFIX + "user-as-default-queue"; line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java File java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@56 PS5, Line 56: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@61 PS5, Line 61: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@80 PS5, Line 80: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@98 PS5, Line 98: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@124 PS5, Line 124: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@156 PS5, Line 156: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@179 PS5, Line 179: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java File java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java: http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@49 PS5, Line 49: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@57 PS5, Line 57: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@59 PS5, Line 59: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@84 PS5, Line 84: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@102 PS5, Line 102: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@107 PS5, Line 107: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@111 PS5, Line 111: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@135 PS5, Line 135: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@141 PS5, Line 141: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@156 PS5, Line 156: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@162 PS5, Line 162: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@165 PS5, Line 165: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@183 PS5, Line 183: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@186 PS5, Line 186: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@202 PS5, Line 202: * Parse xml and instantiate the nested rule line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@234 PS5, Line 234: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@242 PS5, Line 242: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@247 PS5, Line 247: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@263 PS5, Line 263: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@280 PS5, Line 280: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@286 PS5, Line 286: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@303 PS5, Line 303: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@339 PS5, Line 339: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/5/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@345 PS5, Line 345: line has trailing whitespace -- To view, visit http://gerrit.cloudera.org:8080/16500 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I08773f4f9d7cb269b0491080078d6e6f490d8d7a Gerrit-Change-Number: 16500 Gerrit-PatchSet: 5 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Wed, 07 Oct 2020 22:40:57 +0000 Gerrit-HasComments: Yes
