Review Request 68753: SENTRY-2407: SentrySchemaInfo and SQL scripts do not have the new 2.2.0 version

2018-09-18 Thread Na Li via Review Board

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

Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Sergio 
Pena.


Bugs: sentry-2407
https://issues.apache.org/jira/browse/sentry-2407


Repository: sentry


Description
---

increase schema version from 2.1.0 to 2.2.0


Diffs
-

  sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-2.2.0.sql 
PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-2.2.0.sql 
PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-2.2.0.sql 
PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-2.2.0.sql 
PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-2.2.0.sql 
PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-2.1.0-to-2.2.0.sql
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-2.1.0-to-2.2.0.sql
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-2.1.0-to-2.2.0.sql
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-2.1.0-to-2.2.0.sql
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-2.1.0-to-2.2.0.sql
 PRE-CREATION 
  sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.db2 
081f575 
  sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.derby 
a2cc2b6 
  sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.mysql 
a2cc2b6 
  sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.oracle 
a2cc2b6 
  sentry-provider/sentry-provider-db/src/main/resources/upgrade.order.postgres 
a2cc2b6 
  
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreSchemaInfo.java
 f710253 


Diff: https://reviews.apache.org/r/68753/diff/1/


Testing
---


Thanks,

Na Li



Re: Review Request 68716: SENTRY-2403: Incorrect naming in RollingFileWithoutDeleteAppender

2018-09-18 Thread Sergio Pena via Review Board

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


Ship it!




Ship It!

- Sergio Pena


On Sept. 14, 2018, 1:22 p.m., Peter Somogyi wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68716/
> ---
> 
> (Updated Sept. 14, 2018, 1:22 p.m.)
> 
> 
> Review request for sentry.
> 
> 
> Bugs: SENTRY-2403
> https://issues.apache.org/jira/browse/SENTRY-2403
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> RollingFileWithoutDeleteAppender incorrectly append timestamp to logfiles 
> without removing previous one.
> 
> 
> Diffs
> -
> 
>   
> sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/log/appender/RollingFileWithoutDeleteAppender.java
>  fd133f34f 
>   
> sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/log/appender/TestRollingFileWithoutDeleteAppender.java
>  ca9062bca 
> 
> 
> Diff: https://reviews.apache.org/r/68716/diff/1/
> 
> 
> Testing
> ---
> 
> Modified TestRollingFileWithoutDeleteAppender to check the pattern correctly.
> 
> 
> Thanks,
> 
> Peter Somogyi
> 
>