---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59523/#review194153
---
Ship it!
Ship It!
- Alejandro Fernandez
On May 24, 2017, 11
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64323/#review192807
---
Ship it!
Ship It!
- Alejandro Fernandez
On Dec. 5, 2017, 2
ent269880>
Make ATLAS_ENTITIES a constant
tagsync/src/main/java/org/apache/ranger/tagsync/source/atlasrest/AtlasRESTTagSource.java
Line 175 (original), 173 (patched)
<https://reviews.apache.org/r/64042/#comment269881>
Add JavaDoc
- Alejandro Fernandez
On Nov. 25, 2017,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63987/#review191611
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 21, 2017, 4
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63981/#review191571
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 21, 2017, 9
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63921/#review191525
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 17, 2017, 4
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63949/#review191524
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 20, 2017, 1
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63789/#review191374
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 17, 2017, 6
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63870/#review191209
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 16, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63182/#review191208
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 16, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63785/#review190976
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 14, 2017, 4
/KMSAuthenticationFilter.java
Line 74 (original), 73 (patched)
<https://reviews.apache.org/r/63699/#comment268127>
Which other versions of Hadoop are supported?
- Alejandro Fernandez
On Nov. 9, 2017, 1:17 p.m., Colm O hEigeartaigh
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63686/#review190506
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 8, 2017, 11
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63627/#review190346
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 7, 2017, 3
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63461/#review190192
---
Ship it!
Ship It!
- Alejandro Fernandez
On Nov. 1, 2017, 12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63405/#review189634
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 30, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63352/#review189633
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 30, 2017, 6
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63403/#review189630
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 30, 2017, 3
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63404/#review189629
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 30, 2017, 6
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63351/#review189439
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 27, 2017, 9
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62882/#review189112
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 17, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63225/#review189003
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 23, 2017
r.java
Lines 601 (patched)
<https://reviews.apache.org/r/63225/#comment265930>
Add some JavaDoc for this function
- Alejandro Fernandez
On Oct. 23, 2017, 9:45 p.m., Abhay Kulkarni wrote:
>
> ---
> This is an automatic
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63142/#review188855
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 20, 2017, 6
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63145/#review188803
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 20, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63147/#review188714
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 19, 2017, 1
JAVA_VERSION_REQUIRED" ]]; then
...
- Alejandro Fernandez
On Oct. 19, 2017, 1 p.m., pengjianhua wrote:
>
> ---
> This is an automatically generated e-mail. To reply
/HdfsClient.java
Lines 300 (patched)
<https://reviews.apache.org/r/63142/#comment265718>
Doesn't this also have to set
dfs.namenode.http-address.$cluster.$nn_id ?
and potentially https instead if SSL is enabled.
- Alejandro Fernandez
On Oct. 19, 2017, 11:41 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63085/#review188568
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 18, 2017, 7
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63085/#review188547
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 18, 2017, 6
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63115/#review188532
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 18, 2017, 5
/ranger/plugin/contextenricher/RangerTagEnricher.java
Lines 155 (patched)
<https://reviews.apache.org/r/63085/#comment265527>
Please add JavaDoc to new classes & methods.
- Alejandro Fernandez
On Oct. 18, 2017, 1:03 a.m., Abhay Kulk
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62882/#review188343
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 17, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63068/#review188342
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 17, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63055/#review188244
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 16, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62483/#review188190
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 16, 2017, 6
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63030/#review188177
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 16, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62971/#review187987
---
What testing was done?
- Alejandro Fernandez
On Oct. 13, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62967/#review187986
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 13, 2017, 3
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62850/#review187867
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 12, 2017, 8
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62947/#review187858
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 12, 2017, 7
/filter/RangerSecurityContextFormationFilter.java
Lines 144 (patched)
<https://reviews.apache.org/r/62882/#comment264870>
This should include "includeSubDomains"
https://www.owasp.org/index.php/HTTP_Strict_Transport_Security_Cheat_Sheet
- Alejandro Fernandez
On Oct. 12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62918/#review187821
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 12, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62724/#review187613
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 2, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62850/#review187572
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 10, 2017
(patched)
<https://reviews.apache.org/r/62850/#comment264457>
Please add JavaDoc to all new classes and important methods
- Alejandro Fernandez
On Oct. 9, 2017, 9:19 p.m., Endre Zoltan Kovacs wrote:
>
> ---
> This is an automatic
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62686/#review187428
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 9, 2017, 2
/ranger/rest/XUserREST.java
Line 373 (original), 373 (patched)
<https://reviews.apache.org/r/62714/#comment264040>
Should use StringUtils.isNotBlank(userName)
- Alejandro Fernandez
On Oct. 4, 2017, 10:38 a.m., Nitin Galave
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62751/#review186951
---
Ship it!
Ship It!
- Alejandro Fernandez
On Oct. 3, 2017, 2
> On Oct. 2, 2017, 6:13 p.m., Alejandro Fernandez wrote:
> >
Awesome contribution.
- Alejandro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62710/#rev
opConnectorResponse.java
Lines 21 (patched)
<https://reviews.apache.org/r/62710/#comment263706>
Add JavaDoc
- Alejandro Fernandez
On Sept. 30, 2017, 8:34 a.m., Qiang Zhang wrote:
>
> ---
> This is an automatically gen
/XUserREST.java
Line 1094 (original), 1094 (patched)
<https://reviews.apache.org/r/62714/#comment263699>
Does this have to remain backward compatible?
+1 for adding new APIs instead.
- Alejandro Fernandez
On Sept. 30, 2017, 11:26 a.m., Nitin Galave
/apache/ranger/authentication/PasswordValidator.java
Line 117 (original), 117 (patched)
<https://reviews.apache.org/r/62713/#comment263698>
Put a space after the ","
- Alejandro Fernandez
On Sept. 30, 2017, 10:17 a.m., F
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62657/#review186595
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 28, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62657/#review186596
---
Which browsers was this tested on?
- Alejandro Fernandez
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62662/#review186594
---
Any testing done on this?
- Alejandro Fernandez
On Sept. 28
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62659/#review186584
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 28, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62436/#review186486
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 27, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62024/#review186458
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 27, 2017
/PatchForNifiResourceUpdateExclude_J10011.java
Lines 108 (patched)
<https://reviews.apache.org/r/62437/#comment263006>
Might want to add some javadoc
- Alejandro Fernandez
On Sept. 27, 2017, 10:11 a.m., Nikhil P wrote:
>
> ---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62520/#review186451
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 27, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62567/#review186450
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 26, 2017
Nitpick, small typo in "Policies"
- Alejandro Fernandez
On Sept. 27, 2017, 3:52 a.m., bhavik patel wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
>
/ranger/biz/XUserMgr.java
Lines 589 (patched)
<https://reviews.apache.org/r/62555/#comment262822>
Small nitpick here, there are 2 spaces after "groupname ="
- Alejandro Fernandez
On Sept. 26, 2017, 8:35 p.m., Sailaja
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62519/#review186316
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 22, 2017
/ranger/patch/PatchForNifiResourceUpdateExclude_J10010.java
Lines 43 (patched)
<https://reviews.apache.org/r/62436/#comment262770>
Please add some Javadoc for what this class does.
- Alejandro Fernandez
On Sept. 26, 2017, 4:27 p.m., Nikhil P
/ranger/rest/XUserREST.java
Line 358 (original), 367 (patched)
<https://reviews.apache.org/r/62024/#comment262769>
Nitpick here, but should have a space before and after = sign.
Same in line 353 and 382
- Alejandro Fernandez
On Sept. 25, 2017, 5:05 a.m., Nitin Galave
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62398/#review186292
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 21, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62490/#review186290
---
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 26, 2017
check using StringUtils.isNotBlank instead of just null?
security-admin/src/main/java/org/apache/ranger/service/XGroupUserService.java
Line 86 (original), 86 (patched)
<https://reviews.apache.org/r/62555/#comment262635>
Can remove unnecessary var.
- Alejandro Fernandez
On Sept. 26, 2017, 1
potentially regex) of all the resource paths that
should be considered under this type of aggregation?
- Alejandro Fernandez
On Sept. 25, 2017, 11:50 p.m., Alejandro Fernandez wrote:
>
> ---
> This is an automatically generated e-m
[
https://issues.apache.org/jira/browse/RANGER-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16179963#comment-16179963
]
Alejandro Fernandez commented on RANGER-1780:
-
That's a very g
.5AuditConfiguration-Summarization
Thanks,
Alejandro Fernandez
[
https://issues.apache.org/jira/browse/RANGER-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alejandro Fernandez updated RANGER-1780:
Description:
AuditSummaryQueue already has logic to enable the summarization, but
[
https://issues.apache.org/jira/browse/RANGER-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alejandro Fernandez updated RANGER-1780:
Attachment: ranger_summary.png
> Allow AuditSummaryQueue to aggregate events
[
https://issues.apache.org/jira/browse/RANGER-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alejandro Fernandez updated RANGER-1780:
Attachment: RANGER-1780.patch
> Allow AuditSummaryQueue to aggregate events in
[
https://issues.apache.org/jira/browse/RANGER-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alejandro Fernandez updated RANGER-1780:
Description:
AuditSummaryQueue already has logic to enable the summarization, but
[
https://issues.apache.org/jira/browse/RANGER-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Alejandro Fernandez updated RANGER-1780:
Summary: Allow AuditSummaryQueue to aggregate events in the same directory
(was
Alejandro Fernandez created RANGER-1780:
---
Summary: Optimize Audit Logging by Aggregating/Skipping similar
AuthzAuditEvents
Key: RANGER-1780
URL: https://issues.apache.org/jira/browse/RANGER-1780
79 matches
Mail list logo