InvisibleProgrammer commented on PR #4131:
URL: https://github.com/apache/hive/pull/4131#issuecomment-1506552227
I wonder, why it is necessary to check if the tables are not native tables
before getting or setting column statistics?
The interface already provides canSet... and canProvide... methods. Isn't it
enough to call those methods to decide if it can be done?
```java
if (tbl != null && tbl.isNonNative() &&
tbl.getStorageHandler().canSetColStatistics(tbl)) {
tbl.getStorageHandler().setColStatistics(tbl, colStats);
}
if (table.isNonNative() &&
table.getStorageHandler().canProvideColStatistics(table)) {
```
--
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]
