Skip to content

feat!: Support only sending a subset of the fields to OPA #49

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 19 commits into from
Nov 5, 2024
Merged

feat!: Support only sending a subset of the fields to OPA #49

merged 19 commits into from
Nov 5, 2024

Conversation

sbernauer
Copy link
Member

@sbernauer sbernauer commented Oct 25, 2024 •
edited by lfrancke
Loading

Description

One step of working on #48

There is a branch to build a custom image: https://github.com/stackabletech/docker-images/tree/feat/hdfs-opa-performance

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes
# Reviewer
- [x] Code contains useful comments
- [ ] (Integration-)Test cases added
- [ ] Documentation added or updated
- [x] Changelog updated

# Acceptance
- [ ] Proper release label has been added

@sbernauer sbernauer self-assigned this Oct 25, 2024
@sbernauer sbernauer changed the title WIP: First implementation Support only sending a subset of the fields to OPA Oct 25, 2024
@sbernauer sbernauer changed the title Support only sending a subset of the fields to OPA feat!: Support only sending a subset of the fields to OPA Oct 25, 2024
lfrancke
lfrancke requested changes Nov 4, 2024
View reviewed changes
@lfrancke lfrancke mentioned this pull request Nov 4, 2024
Closed
2 tasks
@sbernauer sbernauer requested review from lfrancke and removed request for lfrancke November 4, 2024 16:10
lfrancke
lfrancke requested changes Nov 5, 2024
View reviewed changes
Copy link
Member

@lfrancke lfrancke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only two minor things left

sbernauer and others added 3 commits November 5, 2024 12:56
@sbernauer @lfrancke
Update src/main/java/tech/stackable/hadoop/StackableAccessControlEnfo…
e4df413 
…rcer.java

Co-authored-by: Lars Francke <git@lars-francke.de>
@sbernauer @lfrancke
Update README.md
2caf1cd 
Co-authored-by: Lars Francke <git@lars-francke.de>
@sbernauer
Update benchmark to use nested folder
ead673f
@sbernauer sbernauer requested a review from lfrancke November 5, 2024 11:57
@sbernauer sbernauer added this pull request to the merge queue Nov 5, 2024
Merged via the queue into main with commit 99e728c Nov 5, 2024
13 checks passed
@sbernauer sbernauer deleted the feat/extended-request branch November 5, 2024 12:23
@sbernauer sbernauer mentioned this pull request Nov 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lfrancke lfrancke lfrancke approved these changes

Assignees

@sbernauer sbernauer

Labels
release/24.11.0
Projects
Stackable Engineering
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sbernauer @lfrancke