-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74665/#review225854
-----------------------------------------------------------


Ship it!




Ship It!

- Subhrat Chaudhary


On Oct. 12, 2023, 11:31 a.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/74665/
> -----------------------------------------------------------
> 
> (Updated Oct. 12, 2023, 11:31 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Asit Vadhavkar, Abhay Kulkarni, 
> Mehul Parikh, Monika Kachhadiya, Pradeep Agrawal, Prashant Satam, Ramesh 
> Mani, and Subhrat Chaudhary.
> 
> 
> Bugs: RANGER-4440
>     https://issues.apache.org/jira/browse/RANGER-4440
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> - added new column x_security_zone.gz_jsonData, of type LONGBLOB
> - when Json text compression is enabled, compressed bytes are now stored in 
> the new column. Earlier, compressed bytes were stored in existing column 
> jsonData, which failed for some cases due to character validation at the 
> database level. Storing in blob type column eliminates this issue
> 
> 
> Diffs
> -----
> 
>   
> agents-common/src/main/java/org/apache/ranger/authorization/utils/StringUtil.java
>  12a89889a 
>   security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql 
> dbeeaf423 
>   security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql 
> 065bae0df 
>   
> security-admin/src/main/java/org/apache/ranger/entity/XXSecurityZoneBase.java 
> ecdaf2485 
>   
> security-admin/src/main/java/org/apache/ranger/service/RangerSecurityZoneServiceService.java
>  8acdd9813 
> 
> 
> Diff: https://reviews.apache.org/r/74665/diff/1/
> 
> 
> Testing
> -------
> 
> - verified that Ranger stores and read the compressed data in the new column 
> x_security_zone.gz_jsonData
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>

Reply via email to