Skip to content

[ZEPPELIN-841] Allow note creators to define row level permissions - #878

Closed
prasadwagle wants to merge 1 commit into
apache:masterfrom
prasadwagle:ZEPPELIN-841
Closed

[ZEPPELIN-841] Allow note creators to define row level permissions#878
prasadwagle wants to merge 1 commit into
apache:masterfrom
prasadwagle:ZEPPELIN-841

Conversation

@prasadwagle

Copy link
Copy Markdown

What is this PR for?

Allow note creators to define row level permissions.
Here's an article that describes the use case in tableau:
http://kb.tableau.com/articles/knowledgebase/row-level-security-and-user-filters

What type of PR is it?

Feature

Todos

  • - Code clean up (comments, logging)
  • - Documentation
  • - Unit tests

What is the Jira issue?

ZEPPELIN-841

How should this be tested?

  1. Create a query
  2. Add zeppelin_user column to the query to restrict access for current user to certain rows
  3. Check if only those rows are visible

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update?
    No
  • Is there breaking changes for older versions?
    No
  • Does this needs documentation?
    Yes

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@prasadwagle