Skip to content
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.

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

Fix compilation error in CoordinatorBasicAuthorizerResourceTest #7667

Merged
merged 2 commits into from
May 16, 2019

Conversation

jon-wei
Copy link
Contributor

@jon-wei jon-wei commented May 15, 2019

Fixes a conflict between #7635 and #7661, this wasn't caught as both PRs compiled fine individually had completed their CI runs.

@jon-wei jon-wei added the Bug label May 15, 2019
Copy link
Contributor

@jihoonson jihoonson left a comment

Choose a reason for hiding this comment

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

+1 after CI.

@himanshug
Copy link
Contributor

this was failing my PR build and I was scratching my head :)

@jihoonson jihoonson merged commit 7d63c29 into apache:master May 16, 2019
gianm pushed a commit to implydata/druid-public that referenced this pull request Jul 4, 2019
…he#7667)

* Fix compilation error in CoordinatorBasicAuthorizerResourceTest

* Don't use simplifyPermissions
@clintropolis clintropolis added this to the 0.16.0 milestone Aug 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants