ayushtkn commented on code in PR #4453:
URL: https://github.com/apache/hive/pull/4453#discussion_r1245267217
##########
ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveStorageHandler.java:
##########
@@ -316,14 +315,15 @@ default StorageFormatDescriptor
getStorageFormatDescriptor(Table table) throws S
/**
* Checks whether the table supports appending data files to the table.
* @param table the table
+ * @param withPartClause whether a partition is specified
* @return true if the table can append files directly to the table
* @throws SemanticException in case of any error.
*/
- default boolean supportsAppendData(Table table) throws SemanticException {
+ default boolean supportsAppendData(Table table, boolean withPartClause)
throws SemanticException {
Review Comment:
it isn't skipping part check, it is denote whether the Load has partition
explicitly specified or not
--
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]
