Skip to content

Commit

Permalink
Disable tests broken by Hive breaking changes
Browse files Browse the repository at this point in the history
  • Loading branch information
findepi committed Sep 9, 2019
1 parent 0e8219c commit 6576d1b
Showing 1 changed file with 29 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
import static com.google.common.base.Preconditions.checkState;
import static java.lang.Integer.parseInt;
import static java.util.Objects.requireNonNull;
import static org.assertj.core.api.Assertions.assertThatThrownBy;

public class TestHive
extends AbstractTestHive
Expand Down Expand Up @@ -72,4 +73,32 @@ public void testGetPartitionSplitsTableOfflinePartition()

super.testGetPartitionSplitsTableOfflinePartition();
}

@Override
public void testTypesRcBinary()
throws Exception
{
if (getHiveVersionMajor() >= 3) {
// TODO (https://github.com/prestosql/presto/issues/1218) requires https://issues.apache.org/jira/browse/HIVE-22167
assertThatThrownBy(super::testTypesRcBinary)
.isInstanceOf(AssertionError.class)
.hasMessage("expected [2011-05-06 01:23:09.123] but found [2011-05-06 07:08:09.123]");
return;
}
super.testTypesRcBinary();
}

@Override
public void testTypesParquet()
throws Exception
{
if (getHiveVersionMajor() >= 3) {
// TODO (https://github.com/prestosql/presto/issues/1218) requires https://issues.apache.org/jira/browse/HIVE-21002
assertThatThrownBy(super::testTypesParquet)
.isInstanceOf(AssertionError.class)
.hasMessage("expected [2011-05-06 01:23:09.123] but found [2011-05-06 07:08:09.123]");
return;
}
super.testTypesParquet();
}
}

0 comments on commit 6576d1b

Please sign in to comment.