[ https://issues.apache.org/jira/browse/HIVE-21401?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16788483#comment-16788483 ]
Hive QA commented on HIVE-21401: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12961769/HIVE-21401.05.patch {color:green}SUCCESS:{color} +1 due to 2 test(s) being added or modified. {color:red}ERROR:{color} -1 due to 168 failed/errored test(s), 15757 tests executed *Failed tests:* {noformat} TestCommands - did not produce a TEST-*.xml file (likely timed out) (batchId=204) TestEximReplicationTasks - did not produce a TEST-*.xml file (likely timed out) (batchId=204) TestHCatClient - did not produce a TEST-*.xml file (likely timed out) (batchId=204) TestNoopCommand - did not produce a TEST-*.xml file (likely timed out) (batchId=204) TestReplicationTask - did not produce a TEST-*.xml file (likely timed out) (batchId=204) org.apache.hadoop.hive.cli.TestAccumuloCliDriver.testCliDriver[accumulo_custom_key2] (batchId=267) org.apache.hadoop.hive.cli.TestAccumuloCliDriver.testCliDriver[accumulo_custom_key] (batchId=267) org.apache.hadoop.hive.cli.TestAccumuloCliDriver.testCliDriver[accumulo_index] (batchId=267) org.apache.hadoop.hive.cli.TestAccumuloCliDriver.testCliDriver[accumulo_predicate_pushdown] (batchId=267) org.apache.hadoop.hive.cli.TestAccumuloCliDriver.testCliDriver[accumulo_queries] (batchId=267) org.apache.hadoop.hive.cli.TestAccumuloCliDriver.testCliDriver[accumulo_single_sourced_multi_insert] (batchId=267) org.apache.hadoop.hive.cli.TestBeeLineDriver.testCliDriver[escape_comments] (batchId=275) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[ambiguitycheck] (batchId=79) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[annotate_stats_table] (batchId=23) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[create_union_table] (batchId=29) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[ctas] (batchId=7) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[ctas_colname] (batchId=64) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[ctas_uses_database_location] (batchId=37) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[drop_deleted_partitions] (batchId=72) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[drop_multi_partitions] (batchId=79) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[escape_comments] (batchId=82) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[explain_ddl] (batchId=51) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[fileformat_sequencefile] (batchId=51) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[fileformat_text] (batchId=53) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[groupby_duplicate_key] (batchId=7) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[infer_bucket_sort_reducers_power_two] (batchId=14) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[input10] (batchId=41) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[input15] (batchId=1) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[input1] (batchId=75) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[input2] (batchId=96) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[inputddl1] (batchId=32) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[inputddl2] (batchId=12) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[inputddl3] (batchId=76) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[inputddl6] (batchId=57) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[json_serde1] (batchId=37) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[merge3] (batchId=64) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[mm_iow_temp] (batchId=11) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[nonReservedKeyWords] (batchId=66) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[nonmr_fetch] (batchId=22) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[nullformatCTAS] (batchId=42) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[nullformat] (batchId=47) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[orc_createas1] (batchId=95) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[parallel_orderby] (batchId=59) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[repl_2_exim_basic] (batchId=85) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[repl_3_exim_metadata] (batchId=62) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[serde_opencsv] (batchId=81) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[serde_regex] (batchId=41) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_create_table_alter] (batchId=32) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_create_table_db_table] (batchId=71) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_create_table_delimited] (batchId=31) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_create_table_partitioned] (batchId=40) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_create_table_serde] (batchId=60) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_create_table_temp_table] (batchId=90) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_create_table_view] (batchId=96) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_tables] (batchId=21) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[show_tablestatus] (batchId=84) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[skewjoin_noskew] (batchId=18) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[skewjoin_onesideskew] (batchId=77) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[smb_mapjoin9] (batchId=44) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[symlink_text_input_format] (batchId=51) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[temp_table_truncate] (batchId=28) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[truncate_table] (batchId=87) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[unicode_comments] (batchId=42) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[union25] (batchId=75) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vector_decimal_6] (batchId=15) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vector_tablesample_rows] (batchId=56) org.apache.hadoop.hive.cli.TestContribCliDriver.testCliDriver[fileformat_base64] (batchId=270) org.apache.hadoop.hive.cli.TestContribCliDriver.testCliDriver[serde_regex] (batchId=270) org.apache.hadoop.hive.cli.TestContribNegativeCliDriver.testCliDriver[serde_regex] (batchId=272) org.apache.hadoop.hive.cli.TestHBaseCliDriver.org.apache.hadoop.hive.cli.TestHBaseCliDriver (batchId=103) org.apache.hadoop.hive.cli.TestHBaseCliDriver.org.apache.hadoop.hive.cli.TestHBaseCliDriver (batchId=104) org.apache.hadoop.hive.cli.TestHBaseCliDriver.org.apache.hadoop.hive.cli.TestHBaseCliDriver (batchId=105) org.apache.hadoop.hive.cli.TestHBaseCliDriver.org.apache.hadoop.hive.cli.TestHBaseCliDriver (batchId=106) org.apache.hadoop.hive.cli.TestHBaseCliDriver.org.apache.hadoop.hive.cli.TestHBaseCliDriver (batchId=107) org.apache.hadoop.hive.cli.TestHBaseCliDriver.org.apache.hadoop.hive.cli.TestHBaseCliDriver (batchId=108) org.apache.hadoop.hive.cli.TestMiniDruidCliDriver.testCliDriver[druid_timestamptz2] (batchId=195) org.apache.hadoop.hive.cli.TestMiniDruidCliDriver.testCliDriver[druidmini_dynamic_partition] (batchId=195) org.apache.hadoop.hive.cli.TestMiniDruidCliDriver.testCliDriver[druidmini_mv] (batchId=195) org.apache.hadoop.hive.cli.TestMiniDruidCliDriver.testCliDriver[druidmini_test_alter] (batchId=195) org.apache.hadoop.hive.cli.TestMiniDruidCliDriver.testCliDriver[druidmini_test_insert] (batchId=195) org.apache.hadoop.hive.cli.TestMiniHiveKafkaCliDriver.testCliDriver[kafka_storage_handler] (batchId=275) org.apache.hadoop.hive.cli.TestMiniLlapCliDriver.testCliDriver[external_table_purge] (batchId=157) org.apache.hadoop.hive.cli.TestMiniLlapCliDriver.testCliDriver[rcfile_createas1] (batchId=155) org.apache.hadoop.hive.cli.TestMiniLlapCliDriver.testCliDriver[unionDistinct_1] (batchId=155) org.apache.hadoop.hive.cli.TestMiniLlapCliDriver.testCliDriver[whroot_external1] (batchId=155) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[ctas] (batchId=160) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[dynamic_partition_pruning] (batchId=167) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[explainuser_1] (batchId=168) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[partition_ctas] (batchId=171) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[semijoin_reddedup] (batchId=169) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[strict_managed_tables2] (batchId=178) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[temp_table] (batchId=183) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[tez_dml] (batchId=168) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[tez_join_result_complex] (batchId=162) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[union_top_level] (batchId=176) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vector_char_varchar_1] (batchId=184) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vector_decimal_6] (batchId=162) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vector_windowing_streaming] (batchId=173) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vectorized_dynamic_partition_pruning] (batchId=167) org.apache.hadoop.hive.cli.TestMiniSparkOnYarnCliDriver.testCliDriver[parallel_orderby] (batchId=191) org.apache.hadoop.hive.cli.TestMiniSparkOnYarnCliDriver.testCliDriver[spark_dynamic_partition_pruning] (batchId=190) org.apache.hadoop.hive.cli.TestMiniSparkOnYarnCliDriver.testCliDriver[spark_explainuser_1] (batchId=191) org.apache.hadoop.hive.cli.TestMiniSparkOnYarnCliDriver.testCliDriver[spark_vectorized_dynamic_partition_pruning] (batchId=190) org.apache.hadoop.hive.cli.TestMiniTezCliDriver.testCliDriver[explainanalyze_1] (batchId=109) org.apache.hadoop.hive.cli.TestMiniTezCliDriver.testCliDriver[explainanalyze_3] (batchId=109) org.apache.hadoop.hive.cli.TestMiniTezCliDriver.testCliDriver[explainuser_3] (batchId=109) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[authorization_explain] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[avro_decimal] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[constraint_duplicate_name] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_external_acid] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_not_acid] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_table_wrong_regex] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_view_failure2] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_with_constraints_duplicate_name] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_with_fk_constraint] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_with_fk_pk_same_tab] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_with_fk_uk_same_tab] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_with_fk_wrong_ref2] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[create_with_fk_wrong_ref] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[dbtxnmgr_notablelock] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[dbtxnmgr_notableunlock] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[deletejar] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[describe_xpath1] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[describe_xpath2] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[describe_xpath3] (batchId=101) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[describe_xpath4] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[drop_table_failure2] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[drop_table_used_by_mv] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[drop_view_failure1] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[druid_address] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[druid_buckets] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[druid_case] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[druid_datasource2] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[druid_datasource] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[druid_location] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[druid_partitions] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[external1] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[insert_sorted] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[lockneg1] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[lockneg2] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[lockneg3] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[materialized_view_drop2] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[materialized_view_drop] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[nested_complex_neg] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[serde_regex3] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[serde_regex] (batchId=99) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[special_character_in_tabnames_1] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[strict_managed_tables1] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[strict_managed_tables4] (batchId=100) org.apache.hadoop.hive.cli.TestNegativeCliDriver.testCliDriver[strict_managed_tables5] (batchId=100) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[ctas] (batchId=114) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[skewjoin_noskew] (batchId=119) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[temp_table] (batchId=151) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[union25] (batchId=142) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[union_top_level] (batchId=139) org.apache.hadoop.hive.ql.TestMTQueries.testMTQueries1 (batchId=253) org.apache.hadoop.hive.ql.lockmgr.TestDbTxnManager2.testLockingOnInsertIntoNonNativeTables (batchId=326) org.apache.hadoop.hive.ql.lockmgr.TestDbTxnManager2.testLockingOnInsertOverwriteNonNativeTables (batchId=326) org.apache.hive.hcatalog.cli.TestSemanticAnalysis.testAddPartPass (batchId=209) org.apache.hive.hcatalog.cli.TestSemanticAnalysis.testAddReplaceCols (batchId=209) org.apache.hive.hcatalog.cli.TestSemanticAnalysis.testAlterTableRename (batchId=209) org.apache.hive.hcatalog.cli.TestSemanticAnalysis.testAlterTblClusteredBy (batchId=209) org.apache.hive.hcatalog.cli.TestSemanticAnalysis.testAlterTblTouch (batchId=209) org.apache.hive.hcatalog.cli.TestSemanticAnalysis.testCTLFail (batchId=209) org.apache.hive.hcatalog.cli.TestSemanticAnalysis.testCTLPass (batchId=209) org.apache.hive.hcatalog.cli.TestSemanticAnalysis.testChangeColumns (batchId=209) org.apache.hive.hcatalog.cli.TestUseDatabase.testAlterTablePass (batchId=209) org.apache.hive.jdbc.TestJdbcDriver2.testErrorMessages (batchId=262) {noformat} Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/16423/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/16423/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-16423/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.YetusPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase Tests exited with: TestsFailedException: 168 tests failed {noformat} This message is automatically generated. ATTACHMENT ID: 12961769 - PreCommit-HIVE-Build > Break up DDLTask - extract Table related operations > --------------------------------------------------- > > Key: HIVE-21401 > URL: https://issues.apache.org/jira/browse/HIVE-21401 > Project: Hive > Issue Type: Sub-task > Components: Hive > Affects Versions: 3.1.1 > Reporter: Miklos Gergely > Assignee: Miklos Gergely > Priority: Major > Fix For: 4.0.0 > > Attachments: HIVE-21401.01.patch, HIVE-21401.02.patch, > HIVE-21401.03.patch, HIVE-21401.04.patch, HIVE-21401.05.patch > > > DDLTask is a huge class, more than 5000 lines long. The related DDLWork is > also a huge class, which has a field for each DDL operation it supports. The > goal is to refactor these in order to have everything cut into more > handleable classes under the package org.apache.hadoop.hive.ql.exec.ddl: > * have a separate class for each operation > * have a package for each operation group (database ddl, table ddl, etc), so > the amount of classes under a package is more manageable > * make all the requests (DDLDesc subclasses) immutable > * DDLTask should be agnostic to the actual operations > * right now let's ignore the issue of having some operations handled by > DDLTask which are not actual DDL operations (lock, unlock, desc...) > In the interim time when there are two DDLTask and DDLWork classes in the > code base the new ones in the new package are called DDLTask2 and DDLWork2 > thus avoiding the usage of fully qualified class names where both the old and > the new classes are in use. > Step #2: extract all the table related operations from the old DDLTask except > alter table, and move them under the new package. Also create the new > internal framework. -- This message was sent by Atlassian JIRA (v7.6.3#76005)