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 1:

(27 comments)

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

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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


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


http://gerrit.cloudera.org:8080/#/c/18684/1/tests/authorization/test_ranger.py@1596
PS1, Line 1596: a
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: 1
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: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Vincent Tran <[email protected]>
Gerrit-Comment-Date: Thu, 30 Jun 2022 22:24:44 +0000
Gerrit-HasComments: Yes

Reply via email to