aasha commented on a change in pull request #1529:
URL: https://github.com/apache/hive/pull/1529#discussion_r509991300
##########
File path: common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
##########
@@ -655,6 +643,11 @@ private static void populateLlapDaemonVarsSet(Set<String>
llapDaemonVarsSetLocal
"Provide the maximum number of partitions of a table that will be
batched together during \n"
+ "repl load. All the partitions in a batch will make a single
metastore call to update the metadata. \n"
+ "The data for these partitions will be copied before copying the
metadata batch. "),
+
REPL_LOAD_PARTITIONS_WITH_DATA_COPY_BATCH_SIZE("hive.repl.load.partitions.with.data.copy.batch.size",
+ 1000,
Review comment:
It will increase the line length
##########
File path:
ql/src/java/org/apache/hadoop/hive/ql/exec/repl/bootstrap/load/table/LoadTable.java
##########
@@ -270,10 +269,9 @@ static TableLocationTuple tableLocation(ImportTableDesc
tblDesc, Database parent
Path dataPath = fromURI;
Path tmpPath = tgtPath;
- // if move optimization is enabled, copy the files directly to the target
path. No need to create the staging dir.
+ // if acid tables, copy the files directly to the target path. No need to
create the staging dir.
LoadFileType loadFileType;
- if (replicationSpec.isInReplicationScope() &&
- context.hiveConf.getBoolVar(REPL_ENABLE_MOVE_OPTIMIZATION)) {
+ if (replicationSpec.isInReplicationScope() &&
AcidUtils.isTransactionalTable(table)) {
Review comment:
This is only for the ptests which use non acid managed tables.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
