Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16268 )
Change subject: Add blog post about Ranger ...................................................................... Patch Set 10: (10 comments) http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md File _posts/2020-07-31-fine-grained-authz-ranger.md: http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@9 PS10, Line 9: nit: maybe, add 'along with authentication' ? The context here is that authorization (even a coarse-grained one) doesn't make sense if it's impossible to verify an authenticity of a user. http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@13 PS10, Line 13: controlled nit: enforced? http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@37 PS10, Line 37: PostgreSQL, Microsoft SQL Server, Oracle, MySQL, and SQL Anywhere Just curious: why is this ordering? Is that in the order or 'level' of support or it's an arbitrary order? If the latter, maybe it's worth re-ordering these to be in the alphabetic order? http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@40 PS10, Line 40: As most Hadoop ecosystem projects, including : Ranger, are written in Java, the Ranger plugin base, which needs to be included : in any software that integrates with Ranger, is also available only in Java. I needed to read a few times to parse :) Maybe, simplify this to something like: At the time of writing this post, the Ranger plugin is available only in Java. http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@49 PS10, Line 49: Kudu Masters each nit: maybe just A Kudu Master http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@83 PS10, Line 83: download nit: downloading ? http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@83 PS10, Line 83: Kudu nit: Kudu processes ? http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@92 PS10, Line 92: , nit: drop the comma? http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@178 PS10, Line 178: `-ranger_default_database` flag nit: maybe, mention whose flag is that? http://gerrit.cloudera.org:8080/#/c/16268/10/_posts/2020-07-31-fine-grained-authz-ranger.md@209 PS10, Line 209: ownership nit: table ownership ? -- To view, visit http://gerrit.cloudera.org:8080/16268 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: gh-pages Gerrit-MessageType: comment Gerrit-Change-Id: Iae9a4df40dba8eff7327b76b6d6661e759945e9a Gerrit-Change-Number: 16268 Gerrit-PatchSet: 10 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Greg Solovyev <[email protected]> Gerrit-Comment-Date: Fri, 07 Aug 2020 02:34:22 +0000 Gerrit-HasComments: Yes
