saihemanth-cloudera commented on code in PR #2731:
URL: https://github.com/apache/hive/pull/2731#discussion_r959716618
##########
ql/src/java/org/apache/hadoop/hive/ql/ddl/table/storage/concatenate/AlterTableConcatenateDesc.java:
##########
@@ -44,7 +47,8 @@ public class AlterTableConcatenateDesc implements DDLDesc {
private final TableDesc tableDesc;
public AlterTableConcatenateDesc(TableName tableName, Map<String, String>
partitionSpec, ListBucketingCtx lbCtx,
- Path inputDir, Path outputDir, Class<? extends InputFormat>
inputFormatClass, TableDesc tableDesc) {
+ Path inputDir, Path outputDir, Class<? extends InputFormat>
inputFormatClass, TableDesc tableDesc) throws SemanticException {
+ super(AlterTableType.COMPACT, tableName, partitionSpec, null, false,
false, null);
Review Comment:
The idea behind behind this was:
https://github.com/apache/hive/blob/master/ql/src/java/org/apache/hadoop/hive/ql/ddl/table/storage/concatenate/AlterTableConcatenateAnalyzer.java#L101
I thought Concatenate is major type compaction. I will change this as you
suggested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
