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