Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/16500 )
Change subject: PROTOTYPE: Unify Java code under the $IMPALA_HOME/java directory ...................................................................... Patch Set 2: (87 comments) http://gerrit.cloudera.org:8080/#/c/16500/2/bin/clean-cmake.sh File bin/clean-cmake.sh: http://gerrit.cloudera.org:8080/#/c/16500/2/bin/clean-cmake.sh@32 PS2, Line 32: for loc in "${ROOT_DIR}/ -maxdepth 1" "${ROOT_DIR}/be/" "${ROOT_DIR}/java/" "${ROOT_DIR}/common/"\ ; do line too long (103 > 90) http://gerrit.cloudera.org:8080/#/c/16500/2/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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@30 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@31 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@34 PS2, Line 34: public static void convertAllTypesTable(String inDir, String outDir, String hbaseTableName) { line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@55 PS2, Line 55: } line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@67 PS2, Line 67: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@68 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@72 PS2, Line 72: } line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@73 PS2, Line 73: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@74 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@75 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@76 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@77 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@78 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@79 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@80 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@81 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@82 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@83 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@85 PS2, Line 85: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@86 PS2, Line 86: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@88 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@88 PS2, 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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@89 PS2, Line 89: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@96 PS2, Line 96: convertAllTypesTable("AllTypesError", "HBaseAllTypesError", "functional_hbase.hbasealltypeserror"); line too long (103 > 90) http://gerrit.cloudera.org:8080/#/c/16500/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/CsvToHBaseConverter.java@97 PS2, Line 97: convertAllTypesTable("AllTypesErrorNoNulls", "HBaseAllTypesErrorNoNulls", line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/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/2/java/datagenerator/src/main/java/org/apache/impala/datagenerator/TestDataGenerator.java@46 PS2, Line 46: GregorianCalendar endDate = new GregorianCalendar(DEFAULT_END_YEAR, Calendar.DECEMBER, 31); line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/16500/2/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/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@57 PS2, Line 57: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@86 PS2, Line 86: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@99 PS2, Line 99: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@173 PS2, Line 173: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationConfiguration.java@180 PS2, Line 180: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/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/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@61 PS2, Line 61: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@64 PS2, Line 64: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@80 PS2, Line 80: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@81 PS2, Line 81: // Path to XML file containing allocations. line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@83 PS2, Line 83: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@85 PS2, Line 85: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@88 PS2, Line 88: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@91 PS2, Line 91: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@95 PS2, Line 95: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@99 PS2, Line 99: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@101 PS2, Line 101: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@145 PS2, Line 145: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@153 PS2, Line 153: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@167 PS2, Line 167: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@192 PS2, Line 192: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@196 PS2, Line 196: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@397 PS2, Line 397: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@501 PS2, Line 501: } else if ("queue".endsWith(field.getTagName()) || line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/AllocationFileLoaderService.java@532 PS2, Line 532: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/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/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FSQueueType.java@24 PS2, Line 24: * Represents a leaf queue line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FSQueueType.java@26 PS2, Line 26: LEAF, line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FSQueueType.java@27 PS2, Line 27: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/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/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FairSchedulerConfiguration.java@37 PS2, 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/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/FairSchedulerConfiguration.java@44 PS2, 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/2/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/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@56 PS2, Line 56: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@61 PS2, Line 61: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@80 PS2, Line 80: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@98 PS2, Line 98: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@124 PS2, Line 124: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@156 PS2, Line 156: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementPolicy.java@179 PS2, Line 179: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/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/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@49 PS2, Line 49: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@57 PS2, Line 57: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@59 PS2, Line 59: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@84 PS2, Line 84: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@102 PS2, Line 102: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@107 PS2, Line 107: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@111 PS2, Line 111: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@135 PS2, Line 135: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@141 PS2, Line 141: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@156 PS2, Line 156: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@162 PS2, Line 162: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@165 PS2, Line 165: * line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@183 PS2, Line 183: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@186 PS2, Line 186: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@202 PS2, Line 202: * Parse xml and instantiate the nested rule line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@234 PS2, Line 234: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@242 PS2, Line 242: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@247 PS2, Line 247: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@263 PS2, Line 263: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@280 PS2, Line 280: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@286 PS2, Line 286: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@303 PS2, Line 303: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@339 PS2, Line 339: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/16500/2/java/yarn-extras/src/main/java/org/apache/impala/yarn/server/resourcemanager/scheduler/fair/QueuePlacementRule.java@345 PS2, 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: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Thu, 24 Sep 2020 23:07:22 +0000 Gerrit-HasComments: Yes
