[ https://issues.apache.org/jira/browse/HIVE-20966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16713320#comment-16713320 ]
Hive QA commented on HIVE-20966: -------------------------------- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | || || || || {color:brown} master Compile Tests {color} || | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 1m 17s{color} | {color:blue} Maven dependency ordering for branch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 6m 29s{color} | {color:green} master passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 3m 6s{color} | {color:green} master passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 1m 43s{color} | {color:green} master passed {color} | | {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue} 2m 14s{color} | {color:blue} standalone-metastore/metastore-common in master has 29 extant Findbugs warnings. {color} | | {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue} 0m 30s{color} | {color:blue} common in master has 65 extant Findbugs warnings. {color} | | {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue} 1m 5s{color} | {color:blue} standalone-metastore/metastore-server in master has 184 extant Findbugs warnings. {color} | | {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue} 3m 44s{color} | {color:blue} ql in master has 2311 extant Findbugs warnings. {color} | | {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue} 0m 25s{color} | {color:blue} hcatalog/server-extensions in master has 1 extant Findbugs warnings. {color} | | {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue} 0m 35s{color} | {color:blue} itests/hive-unit in master has 2 extant Findbugs warnings. {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 47s{color} | {color:green} master passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 25s{color} | {color:blue} Maven dependency ordering for patch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 3m 31s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 3m 7s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 3m 7s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} checkstyle {color} | {color:red} 0m 15s{color} | {color:red} common: The patch generated 1 new + 427 unchanged - 0 fixed = 428 total (was 427) {color} | | {color:red}-1{color} | {color:red} checkstyle {color} | {color:red} 0m 44s{color} | {color:red} ql: The patch generated 55 new + 854 unchanged - 11 fixed = 909 total (was 865) {color} | | {color:red}-1{color} | {color:red} checkstyle {color} | {color:red} 0m 22s{color} | {color:red} itests/hive-unit: The patch generated 145 new + 658 unchanged - 58 fixed = 803 total (was 716) {color} | | {color:red}-1{color} | {color:red} whitespace {color} | {color:red} 0m 0s{color} | {color:red} The patch has 16 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 1m 15s{color} | {color:red} standalone-metastore/metastore-server generated 4 new + 184 unchanged - 0 fixed = 188 total (was 184) {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 4m 3s{color} | {color:red} ql generated 1 new + 2310 unchanged - 1 fixed = 2311 total (was 2311) {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 0m 33s{color} | {color:red} hcatalog/server-extensions generated 2 new + 1 unchanged - 0 fixed = 3 total (was 1) {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 50s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 12s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 47m 20s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | FindBugs | module:standalone-metastore/metastore-server | | | Boxed value is unboxed and then immediately reboxed in org.apache.hadoop.hive.metastore.messaging.json.JSONAlterPartitionMessage.getWriteId() At JSONAlterPartitionMessage.java:then immediately reboxed in org.apache.hadoop.hive.metastore.messaging.json.JSONAlterPartitionMessage.getWriteId() At JSONAlterPartitionMessage.java:[line 149] | | | Boxed value is unboxed and then immediately reboxed in org.apache.hadoop.hive.metastore.messaging.json.JSONAlterTableMessage.getWriteId() At JSONAlterTableMessage.java:then immediately reboxed in org.apache.hadoop.hive.metastore.messaging.json.JSONAlterTableMessage.getWriteId() At JSONAlterTableMessage.java:[line 124] | | | org.apache.hadoop.hive.metastore.txn.TxnHandler.updateReplId(Connection, ReplLastIdInfo) passes a nonconstant String to an execute or addBatch method on an SQL statement At TxnHandler.java:String to an execute or addBatch method on an SQL statement At TxnHandler.java:[line 944] | | | A prepared statement is generated from a nonconstant String in org.apache.hadoop.hive.metastore.txn.TxnHandler.updateReplId(Connection, ReplLastIdInfo) At TxnHandler.java:from a nonconstant String in org.apache.hadoop.hive.metastore.txn.TxnHandler.updateReplId(Connection, ReplLastIdInfo) At TxnHandler.java:[line 989] | | FindBugs | module:ql | | | Switch statement found in org.apache.hadoop.hive.ql.util.HiveStrictManagedMigration.migrateToManagedTable(Table, TableType, boolean, HiveStrictManagedMigration$HiveUpdater, IMetaStoreClient, Configuration) where one case falls through to the next case At HiveStrictManagedMigration.java:TableType, boolean, HiveStrictManagedMigration$HiveUpdater, IMetaStoreClient, Configuration) where one case falls through to the next case At HiveStrictManagedMigration.java:[lines 757-760] | | FindBugs | module:hcatalog/server-extensions | | | Boxed value is unboxed and then immediately reboxed in org.apache.hive.hcatalog.messaging.json.JSONAlterPartitionMessage.getWriteId() At JSONAlterPartitionMessage.java:then immediately reboxed in org.apache.hive.hcatalog.messaging.json.JSONAlterPartitionMessage.getWriteId() At JSONAlterPartitionMessage.java:[line 116] | | | Boxed value is unboxed and then immediately reboxed in org.apache.hive.hcatalog.messaging.json.JSONAlterTableMessage.getWriteId() At JSONAlterTableMessage.java:then immediately reboxed in org.apache.hive.hcatalog.messaging.json.JSONAlterTableMessage.getWriteId() At JSONAlterTableMessage.java:[line 103] | \\ \\ || Subsystem || Report/Notes || | Optional Tests | asflicense javac javadoc findbugs checkstyle compile | | uname | Linux hiveptest-server-upstream 3.16.0-4-amd64 #1 SMP Debian 3.16.36-1+deb8u1 (2016-09-03) x86_64 GNU/Linux | | Build tool | maven | | Personality | /data/hiveptest/working/yetus_PreCommit-HIVE-Build-15216/dev-support/hive-personality.sh | | git revision | master / aae392f | | Default Java | 1.8.0_111 | | findbugs | v3.0.0 | | checkstyle | http://104.198.109.242/logs//PreCommit-HIVE-Build-15216/yetus/diff-checkstyle-common.txt | | checkstyle | http://104.198.109.242/logs//PreCommit-HIVE-Build-15216/yetus/diff-checkstyle-ql.txt | | checkstyle | http://104.198.109.242/logs//PreCommit-HIVE-Build-15216/yetus/diff-checkstyle-itests_hive-unit.txt | | whitespace | http://104.198.109.242/logs//PreCommit-HIVE-Build-15216/yetus/whitespace-eol.txt | | findbugs | http://104.198.109.242/logs//PreCommit-HIVE-Build-15216/yetus/new-findbugs-standalone-metastore_metastore-server.html | | findbugs | http://104.198.109.242/logs//PreCommit-HIVE-Build-15216/yetus/new-findbugs-ql.html | | findbugs | http://104.198.109.242/logs//PreCommit-HIVE-Build-15216/yetus/new-findbugs-hcatalog_server-extensions.html | | modules | C: standalone-metastore/metastore-common common standalone-metastore/metastore-server ql hcatalog/server-extensions itests/hive-unit U: . | | Console output | http://104.198.109.242/logs//PreCommit-HIVE-Build-15216/yetus.txt | | Powered by | Apache Yetus http://yetus.apache.org | This message was automatically generated. > Support bootstrap and incremental replication to a target with > hive.strict.managed.tables enabled. > -------------------------------------------------------------------------------------------------- > > Key: HIVE-20966 > URL: https://issues.apache.org/jira/browse/HIVE-20966 > Project: Hive > Issue Type: Sub-task > Components: repl > Affects Versions: 4.0.0 > Reporter: mahesh kumar behera > Assignee: mahesh kumar behera > Priority: Major > Labels: DR > Attachments: HIVE-20966.01.patch, HIVE-20966.02.patch, > HIVE-20966.03.patch, HIVE-20966.04.patch, HIVE-20966.04.patch > > > *Requirements:* > Hive2 supports replication of managed tables. But in Hive3 with > hive.strict.managed.tables=true, some of these managed tables are converted > to ACID or MM tables. Also, some of them are converted to external tables > based on below rules. > - Avro format with external schema, Storage handlers, List bucketed tabled > are converted to external tables. > - Location not owned by "hive" user are converted to external table. > - Hive owned ORC format are converted to full ACID transactional table. > - Hive owned Non-ORC format are converted to MM transactional table. > REPL LOAD should apply these rules during bootstrap and incremental phases > and convert the tables accordingly. -- This message was sent by Atlassian JIRA (v7.6.3#76005)