Skip to content

Commit

Permalink
Try moving subclasses of BaseTestImpersonation to EasyOutOfMemory run.
Browse files Browse the repository at this point in the history
  • Loading branch information
jnturton committed Aug 15, 2023
1 parent 976e91c commit 6e7fd59
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
package org.apache.drill.exec.impersonation;

import org.apache.drill.shaded.guava.com.google.common.collect.Maps;
import org.apache.drill.categories.EasyOutOfMemory;
import org.apache.drill.categories.SecurityTest;
import org.apache.drill.exec.store.dfs.WorkspaceConfig;
import org.apache.drill.categories.SlowTest;
Expand All @@ -32,7 +33,7 @@
* the tests. Running this mini cluster is slow and it is best for these tests to only cover
* necessary cases.
*/
@Category({SlowTest.class, SecurityTest.class})
@Category({SlowTest.class, SecurityTest.class, EasyOutOfMemory.class})
public class TestImpersonationDisabledWithMiniDFS extends BaseTestImpersonation {

@BeforeClass
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
*/
package org.apache.drill.exec.impersonation;

import org.apache.drill.categories.EasyOutOfMemory;
import org.apache.drill.categories.SecurityTest;
import org.apache.drill.categories.SlowTest;
import org.apache.drill.categories.UnlikelyTest;
Expand Down Expand Up @@ -51,7 +52,7 @@
/**
* Tests impersonation on metadata related queries as SHOW FILES, SHOW TABLES, CREATE VIEW, CREATE TABLE and DROP TABLE
*/
@Category({SlowTest.class, SecurityTest.class})
@Category({SlowTest.class, SecurityTest.class, EasyOutOfMemory.class})
public class TestImpersonationMetadata extends BaseTestImpersonation {
private static final String user1 = "drillTestUser1";
private static final String user2 = "drillTestUser2";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
*/
package org.apache.drill.exec.impersonation;

import org.apache.drill.categories.EasyOutOfMemory;
import org.apache.drill.categories.SecurityTest;
import org.apache.drill.categories.SlowTest;
import org.apache.drill.common.exceptions.UserRemoteException;
Expand Down Expand Up @@ -45,7 +46,7 @@
* Test queries involving direct impersonation and multilevel impersonation including join queries where each side is
* a nested view.
*/
@Category({SlowTest.class, SecurityTest.class})
@Category({SlowTest.class, SecurityTest.class, EasyOutOfMemory.class})
public class TestImpersonationQueries extends BaseTestImpersonation {
@BeforeClass
public static void setup() throws Exception {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
*/
package org.apache.drill.exec.impersonation;

import org.apache.drill.categories.EasyOutOfMemory;
import org.apache.drill.categories.SecurityTest;
import org.apache.drill.categories.SlowTest;
import org.apache.drill.common.config.DrillProperties;
Expand All @@ -40,7 +41,7 @@
import static org.apache.drill.exec.rpc.user.security.testing.UserAuthenticatorTestImpl.PROCESS_USER_PASSWORD;
import static org.apache.drill.exec.rpc.user.security.testing.UserAuthenticatorTestImpl.TYPE;

@Category({SlowTest.class, SecurityTest.class})
@Category({SlowTest.class, SecurityTest.class, EasyOutOfMemory.class})
public class TestInboundImpersonation extends BaseTestImpersonation {

public static final String OWNER = org1Users[0];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@

import org.apache.drill.shaded.guava.com.google.common.base.Charsets;
import org.apache.drill.shaded.guava.com.google.common.io.Files;
import org.apache.drill.categories.EasyOutOfMemory;
import org.apache.drill.categories.SecurityTest;
import org.apache.drill.common.util.DrillFileUtils;
import org.apache.drill.exec.ExecConstants;
Expand All @@ -33,7 +34,7 @@

import static org.junit.Assert.assertEquals;

@Category(SecurityTest.class)
@Category(SecurityTest.class, EasyOutOfMemory.class)
public class TestInboundImpersonationPrivileges extends BaseTestImpersonation {
private static final org.slf4j.Logger logger =
org.slf4j.LoggerFactory.getLogger(TestInboundImpersonationPrivileges.class);
Expand Down

0 comments on commit 6e7fd59

Please sign in to comment.