veghlaci05 commented on code in PR #3172: URL: https://github.com/apache/hive/pull/3172#discussion_r857385048
########## ql/src/test/queries/clientpositive/acid_insert_overwrite_update.q: ########## @@ -26,7 +26,6 @@ insert overwrite table sequential_update values(current_timestamp, 0, current_ti delete from sequential_update where seq=2; select distinct IF(seq==0, 'LOOKS OKAY', 'BROKEN'), regexp_extract(INPUT__FILE__NAME, '.*/(.*)/[^/]*', 1) from sequential_update; -alter table sequential_update compact 'major'; Review Comment: It turned out that the Q test running environment doesn't start the HMS background threads at all. As a result the issued compactions are never processed. From now on it is not possible to initiate a second compaction on the same table with the same write id before the previous one is cleaned up. As a result, the second compaction request was refused in this test. BTW this questions the necessity of the compaction commands in these tests for me, but I only removed only the second one to make the test green again. -- 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]
