You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Vamsee Yarlagadda <va...@cloudera.com> on 2016/12/16 20:41:40 UTC

Review Request 54819: SENTRY-1526: Sentry processed stays alive after being killed (Fixing Mismerge) (Alexander Kolbasov via Vamsee Yarlagadda)

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

Review request for sentry and Alexander Kolbasov.


Repository: sentry


Description
-------

SENTRY-1526: Sentry processed stays alive after being killed (Fixing Mismerge) (Alexander Kolbasov via Vamsee Yarlagadda)


Diffs
-----

  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java 5e36e48eb5f47dc82b168eca914499ddef6b6516 

Diff: https://reviews.apache.org/r/54819/diff/


Testing
-------

```bash
[INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 approved: 1 licence.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Sentry ............................................ SUCCESS [2.873s]
[INFO] Sentry Core ....................................... SUCCESS [0.206s]
[INFO] Sentry Core Common ................................ SUCCESS [4.610s]
[INFO] Sentry Core Model DB .............................. SUCCESS [1.375s]
[INFO] Sentry Core Model Indexer ......................... SUCCESS [0.777s]
[INFO] Sentry Core Model Search .......................... SUCCESS [0.676s]
[INFO] Sentry Core Model Sqoop ........................... SUCCESS [0.543s]
[INFO] Sentry Core Model Kafka ........................... SUCCESS [0.553s]
[INFO] Sentry Policies ................................... SUCCESS [0.098s]
[INFO] Sentry Policy Common .............................. SUCCESS [0.773s]
[INFO] Sentry Providers .................................. SUCCESS [0.135s]
[INFO] Sentry Provider Common ............................ SUCCESS [0.709s]
[INFO] Sentry Provider File .............................. SUCCESS [0.716s]
[INFO] Sentry Policy Engine .............................. SUCCESS [0.308s]
[INFO] Sentry Bindings ................................... SUCCESS [0.139s]
[INFO] Sentry Hive Binding Configuration ................. SUCCESS [1.334s]
[INFO] Hive follower for Sentry .......................... SUCCESS [1.517s]
[INFO] Sentry HDFS ....................................... SUCCESS [0.098s]
[INFO] Sentry HDFS Common ................................ SUCCESS [4.235s]
[INFO] Sentry Provider DB ................................ SUCCESS [11.961s]
[INFO] Sentry Binding for Kafka .......................... SUCCESS [1.651s]
[INFO] Sentry Provider Cache ............................. SUCCESS [0.485s]
[INFO] Sentry Hive Binding Common ........................ SUCCESS [1.948s]
[INFO] Sentry Binding for Solr ........................... SUCCESS [1.206s]
[INFO] Sentry Binding for Sqoop .......................... SUCCESS [1.194s]
[INFO] Sentry Binding for Hive ........................... SUCCESS [1.446s]
[INFO] Sentry Policy for Indexer ......................... SUCCESS [0.549s]
[INFO] Sentry Solr ....................................... SUCCESS [0.066s]
[INFO] Solr Sentry Core .................................. SUCCESS [1.430s]
[INFO] Solr Sentry handler ............................... SUCCESS [1.613s]
[INFO] Sentry Tests ...................................... SUCCESS [0.194s]
[INFO] Sentry Solr Tests ................................. SUCCESS [3.024s]
[INFO] Sentry Sqoop Tests ................................ SUCCESS [2.447s]
[INFO] Sentry Kafka Tests ................................ SUCCESS [1.427s]
[INFO] Sentry HDFS Service ............................... SUCCESS [1.463s]
[INFO] Sentry HDFS Namenode Plugin ....................... SUCCESS [1.122s]
[INFO] Sentry Hive Tests ................................. SUCCESS [3.560s]
[INFO] Sentry HDFS Dist .................................. SUCCESS [1.251s]
[INFO] Sentry Distribution ............................... SUCCESS [16.520s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1:17.581s
[INFO] Finished at: Fri Dec 16 12:35:23 PST 2016
[INFO] Final Memory: 183M/1509M
[INFO] ------------------------------------------------------------------------
```


Thanks,

Vamsee Yarlagadda


Re: Review Request 54819: SENTRY-1526: Sentry processed stays alive after being killed (Fixing Mismerge) (Alexander Kolbasov via Vamsee Yarlagadda)

Posted by Alexander Kolbasov <ak...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54819/#review159491
-----------------------------------------------------------


Ship it!




Ship It!

- Alexander Kolbasov


On Dec. 16, 2016, 8:41 p.m., Vamsee Yarlagadda wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54819/
> -----------------------------------------------------------
> 
> (Updated Dec. 16, 2016, 8:41 p.m.)
> 
> 
> Review request for sentry and Alexander Kolbasov.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> SENTRY-1526: Sentry processed stays alive after being killed (Fixing Mismerge) (Alexander Kolbasov via Vamsee Yarlagadda)
> 
> 
> Diffs
> -----
> 
>   sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java 5e36e48eb5f47dc82b168eca914499ddef6b6516 
> 
> Diff: https://reviews.apache.org/r/54819/diff/
> 
> 
> Testing
> -------
> 
> ```bash
> [INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 approved: 1 licence.
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Sentry ............................................ SUCCESS [2.873s]
> [INFO] Sentry Core ....................................... SUCCESS [0.206s]
> [INFO] Sentry Core Common ................................ SUCCESS [4.610s]
> [INFO] Sentry Core Model DB .............................. SUCCESS [1.375s]
> [INFO] Sentry Core Model Indexer ......................... SUCCESS [0.777s]
> [INFO] Sentry Core Model Search .......................... SUCCESS [0.676s]
> [INFO] Sentry Core Model Sqoop ........................... SUCCESS [0.543s]
> [INFO] Sentry Core Model Kafka ........................... SUCCESS [0.553s]
> [INFO] Sentry Policies ................................... SUCCESS [0.098s]
> [INFO] Sentry Policy Common .............................. SUCCESS [0.773s]
> [INFO] Sentry Providers .................................. SUCCESS [0.135s]
> [INFO] Sentry Provider Common ............................ SUCCESS [0.709s]
> [INFO] Sentry Provider File .............................. SUCCESS [0.716s]
> [INFO] Sentry Policy Engine .............................. SUCCESS [0.308s]
> [INFO] Sentry Bindings ................................... SUCCESS [0.139s]
> [INFO] Sentry Hive Binding Configuration ................. SUCCESS [1.334s]
> [INFO] Hive follower for Sentry .......................... SUCCESS [1.517s]
> [INFO] Sentry HDFS ....................................... SUCCESS [0.098s]
> [INFO] Sentry HDFS Common ................................ SUCCESS [4.235s]
> [INFO] Sentry Provider DB ................................ SUCCESS [11.961s]
> [INFO] Sentry Binding for Kafka .......................... SUCCESS [1.651s]
> [INFO] Sentry Provider Cache ............................. SUCCESS [0.485s]
> [INFO] Sentry Hive Binding Common ........................ SUCCESS [1.948s]
> [INFO] Sentry Binding for Solr ........................... SUCCESS [1.206s]
> [INFO] Sentry Binding for Sqoop .......................... SUCCESS [1.194s]
> [INFO] Sentry Binding for Hive ........................... SUCCESS [1.446s]
> [INFO] Sentry Policy for Indexer ......................... SUCCESS [0.549s]
> [INFO] Sentry Solr ....................................... SUCCESS [0.066s]
> [INFO] Solr Sentry Core .................................. SUCCESS [1.430s]
> [INFO] Solr Sentry handler ............................... SUCCESS [1.613s]
> [INFO] Sentry Tests ...................................... SUCCESS [0.194s]
> [INFO] Sentry Solr Tests ................................. SUCCESS [3.024s]
> [INFO] Sentry Sqoop Tests ................................ SUCCESS [2.447s]
> [INFO] Sentry Kafka Tests ................................ SUCCESS [1.427s]
> [INFO] Sentry HDFS Service ............................... SUCCESS [1.463s]
> [INFO] Sentry HDFS Namenode Plugin ....................... SUCCESS [1.122s]
> [INFO] Sentry Hive Tests ................................. SUCCESS [3.560s]
> [INFO] Sentry HDFS Dist .................................. SUCCESS [1.251s]
> [INFO] Sentry Distribution ............................... SUCCESS [16.520s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 1:17.581s
> [INFO] Finished at: Fri Dec 16 12:35:23 PST 2016
> [INFO] Final Memory: 183M/1509M
> [INFO] ------------------------------------------------------------------------
> ```
> 
> 
> Thanks,
> 
> Vamsee Yarlagadda
> 
>