Re: Review Request 61889: SENTRY-1895: Sentry should handle the case of multiple notifications with the same ID

2017-08-25 Thread Sergio Pena
> On Aug. 25, 2017, 12:05 a.m., Alexander Kolbasov wrote: > > I think that it is more useful to get SHA1 on the original HMS event (JSON > > object). This will allow us to quickly commpare whether the new > > notification from HMS is the same as the one we already processed or not. > > Otherwi

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-25 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61793/ --- (Updated Aug. 25, 2017, 7:25 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 61889: SENTRY-1895: Sentry should handle the case of multiple notifications with the same ID

2017-08-25 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61889/ --- (Updated Aug. 25, 2017, 7:25 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 61793: SENTRY-1894: Update field size in package.jdo for dataNucleus to match size in sql

2017-08-25 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61793/#review183869 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 61889: SENTRY-1895: Sentry should handle the case of multiple notifications with the same ID

2017-08-25 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61889/ --- (Updated Aug. 25, 2017, 8:34 p.m.) Review request for sentry, Alexander Kolbaso