Skip to content

Commit 282e2c4

Browse files
committed
Revert Accumulo Hadoop 3 experimental changes
1 parent 68c30e3 commit 282e2c4

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

.github/workflows/tests.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,5 +92,5 @@ jobs:
9292
export MAVEN_OPTS="${MAVEN_INSTALL_OPTS}"
9393
./mvnw install ${MAVEN_FAST_INSTALL} -am -pl $(echo '${{ matrix.modules }}' | cut -d' ' -f1)
9494
- name: Maven Tests
95-
if: needs.changes.outputs.codechange == 'true' && ${{ matrix.modules }} == ':presto-accumulo'
96-
run: ./mvnw test ${MAVEN_TEST} -pl ${{ matrix.modules }} -Dhadoop.profile=3
95+
if: needs.changes.outputs.codechange == 'true'
96+
run: ./mvnw test ${MAVEN_TEST} -pl ${{ matrix.modules }}

presto-accumulo/src/test/java/com/facebook/presto/accumulo/AccumuloQueryRunner.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@
3939
import java.io.File;
4040
import java.io.IOException;
4141
import java.nio.file.Files;
42-
import java.util.HashMap;
4342
import java.util.Map;
4443

4544
import static com.facebook.presto.accumulo.AccumuloErrorCode.MINI_ACCUMULO;
@@ -197,9 +196,7 @@ private static MiniAccumuloCluster createMiniAccumuloCluster()
197196
MiniAccumuloCluster accumulo = new MiniAccumuloCluster(macDir, MAC_PASSWORD);
198197
accumulo.getConfig().setDefaultMemory(512, MEGABYTE);
199198
setConfigClassPath(accumulo.getConfig());
200-
Map<String, String> coresiteMap = new HashMap<>();
201-
coresiteMap.put("fs.defaultFS", "file:///");
202-
accumulo.getConfig().setSiteConfig(coresiteMap);
199+
203200
accumulo.start();
204201

205202
// Add shutdown hook to stop MAC and cleanup temporary files

0 commit comments

Comments
 (0)