Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18684 )

Change subject: IMPALA-10122 (Part 2): Allow accessing views created by 
non-superusers
......................................................................


Patch Set 2:

(43 comments)

http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py
File tests/authorization/test_ranger.py:

http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@785
PS2, Line 785: d
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@827
PS2, Line 827: @
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1574
PS2, Line 1574: @
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1575
PS2, Line 1575: @
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1577
PS2, Line 1577: d
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1580
PS2, Line 1580: @
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1581
PS2, Line 1581: @
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1584
PS2, Line 1584: d
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1587
PS2, Line 1587: d
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1605
PS2, Line 1605: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1606
PS2, Line 1606: s
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1608
PS2, Line 1608: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1610
PS2, Line 1610: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1612
PS2, Line 1612: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1614
PS2, Line 1614: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1616
PS2, Line 1616: r
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1618
PS2, Line 1618: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1621
PS2, Line 1621: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1623
PS2, Line 1623: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1624
PS2, Line 1624: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1625
PS2, Line 1625: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1626
PS2, Line 1626: r
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1628
PS2, Line 1628: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1631
PS2, Line 1631: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1633
PS2, Line 1633: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1635
PS2, Line 1635: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1636
PS2, Line 1636: s
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1639
PS2, Line 1639: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1640
PS2, Line 1640: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1641
PS2, Line 1641: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1642
PS2, Line 1642: T
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1643
PS2, Line 1643: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1645
PS2, Line 1645: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1646
PS2, Line 1646: #
flake8: E114 indentation is not a multiple of four (comment)


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1647
PS2, Line 1647: r
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1649
PS2, Line 1649: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1652
PS2, Line 1652: s
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1654
PS2, Line 1654: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1656
PS2, Line 1656: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1658
PS2, Line 1658: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1660
PS2, Line 1660: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1662
PS2, Line 1662: a
flake8: E111 indentation is not a multiple of four


http://gerrit.cloudera.org:8080/#/c/18684/2/tests/authorization/test_ranger.py@1664
PS2, Line 1664: T
flake8: E111 indentation is not a multiple of four



--
To view, visit http://gerrit.cloudera.org:8080/18684
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I50a50931c6eeb0feec28c30531b09269622e6aad
Gerrit-Change-Number: 18684
Gerrit-PatchSet: 2
Gerrit-Owner: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Aman Sinha <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Vincent Tran <[email protected]>
Gerrit-Comment-Date: Sat, 02 Jul 2022 00:54:35 +0000
Gerrit-HasComments: Yes

Reply via email to