deniskuzZ commented on code in PR #5498:
URL: https://github.com/apache/hive/pull/5498#discussion_r1928930444
##########
ql/src/java/org/apache/hadoop/hive/ql/ddl/table/info/desc/DescTableOperation.java:
##########
@@ -190,16 +192,16 @@ private void getColumnsNoColumnPath(Table table,
Partition partition, List<Field
private void getColumnDataColPathSpecified(Table table, Partition part,
List<FieldSchema> cols,
List<ColumnStatisticsObj> colStats, Deserializer deserializer)
- throws SemanticException, HiveException, MetaException {
+ throws HiveException, MetaException {
// when column name is specified in describe table DDL, colPath will be
db_name.table_name.column_name
String colName = desc.getColumnPath().split("\\.")[2];
List<String> colNames = Lists.newArrayList(colName.toLowerCase());
TableName tableName = HiveTableName.of(desc.getDbTableName());
if (null == part) {
- if (table.isPartitioned()) {
+ if (table.isPartitioned() && !table.alwaysUnpartitioned()) {
Review Comment:
@abstractdog, renamed to `hasNonNativePartitionSupport` since we already
have `isNonNative` + nonNative means via storageHanlder
````
public boolean hasNonNativePartitionSupport() {
return getStorageHandler() != null &&
getStorageHandler().supportsPartitioning();
}
````
--
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]
