Skip to content

Commit cfd7fd9

Browse files
committed
revert earlier change re. nsamed libs as dependencies are needed
1 parent 61ca906 commit cfd7fd9

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ Group mappings are resolved on the NameNode and the following configuration shou
4747

4848
#### HADOOP_CLASSPATH
4949

50-
* Fixed value of `"/stackable/hadoop/share/hadoop/tools/lib/hdfs-group-mapper-0.1.0-SNAPSHOT.jar"`
50+
* Fixed value of `"/stackable/hadoop/share/hadoop/tools/lib/*.jar"`
5151

5252
### configOverrides / `core-site.xml`
5353

src/main/java/tech/stackable/hadoop/StackableGroupMapper.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,9 @@ public StackableGroupMapper() {
5858
throw new RuntimeException("Config \"" + OPA_MAPPING_GROUP_NAME_PROP + "\" missing");
5959
}
6060

61+
LOG.info("OPA mapping URL [{}]", opaMappingUrl);
62+
LOG.info("OPA mapping group [{}]", mappingGroupName);
63+
6164
this.json = new ObjectMapper()
6265
// https://github.com/stackabletech/trino-opa-authorizer/issues/24
6366
// OPA server can send other fields, such as `decision_id`` when enabling decision logs

test/stack/10-hdfs.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ spec:
3838
roleGroups:
3939
default:
4040
envOverrides:
41-
HADOOP_CLASSPATH: "/stackable/hadoop/share/hadoop/tools/lib/hdfs-group-mapper-0.1.0-SNAPSHOT.jar"
41+
HADOOP_CLASSPATH: "/stackable/hadoop/share/hadoop/tools/lib/*.jar"
4242
configOverrides:
4343
core-site.xml:
4444
# the mapper is only handled on the namenode so no need to apply this config to all roles

0 commit comments

Comments
 (0)