pvargacl commented on a change in pull request #1532:
URL: https://github.com/apache/hive/pull/1532#discussion_r498204987
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java
##########
@@ -114,16 +114,16 @@
public class AcidUtils {
// This key will be put in the conf file when planning an acid operation
public static final String CONF_ACID_KEY = "hive.doing.acid";
- public static final String BASE_PREFIX = "base_";
+ public static final String BASE_PREFIX = AcidConstants.BASE_PREFIX;
Review comment:
I am not sure I want to remove these public constants, the AcidUtils is
unfortunately is being used by spark-acid and it might break compatibility...
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
