Re: Review Request 45260: SENTRY-1159: Decouple datanucleus dependences for hive-binding V1 and V2

[Colin Ma]

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


Ship it!




Ship It!

- Colin Ma


On March 24, 2016, 5:41 a.m., Dapeng Sun wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45260/
> ---
> 
> (Updated March 24, 2016, 5:41 a.m.)
> 
> 
> Review request for sentry and Colin Ma.
> 
> 
> Bugs: SENTRY-1159
> https://issues.apache.org/jira/browse/SENTRY-1159
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> Since Hive 2.0.0 depends on datanuclues 4+, we should decouple datanuclues 
> dependence on binding.
> 
> 
> Diffs
> -
> 
>   pom.xml d25c314 
>   sentry-binding/sentry-binding-hive/pom.xml fb5f214 
>   sentry-provider/sentry-provider-db/pom.xml bf4dfdc 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
>  c5c5ffb 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
>  94bd2a9 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestSentryRole.java
>  7b4d3b8 
>   sentry-tests/pom.xml 88a28bb 
>   sentry-tests/sentry-tests-hive-v2/pom.xml PRE-CREATION 
>   sentry-tests/sentry-tests-hive/pom.xml 472cce7 
>   sentry-tests/sentry-tests-kafka/pom.xml 54c7205 
>   sentry-tests/sentry-tests-solr/pom.xml e90ca06 
>   sentry-tests/sentry-tests-sqoop/pom.xml 3989ced 
> 
> Diff: https://reviews.apache.org/r/45260/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Dapeng Sun
> 
>

Re: Review Request 45260: SENTRY-1159: Decouple datanucleus dependences for hive-binding V1 and V2

[Dapeng Sun]

> On 三月 24, 2016, 10:04 a.m., Colin Ma wrote:
> > Thanks for the patch, I have 2 comments:
> > 1. Different versions for datanucleus are included for hive-v1 and hive-v2. 
> > When package Sentry, the lib will include all these jars, how to avoid the 
> > conflict?
> > 2. test-hive-v1 is enable as default, when do the pre-commit, how to test 
> > test-hive-v2?

Hi Colin,
Here are my comments:
1. Different versions for datanucleus are included for hive-v1 and hive-v2. 
When package Sentry, the lib will include all these jars, how to avoid the 
conflict?
Currently hive-1 will package, after 
https://issues.apache.org/jira/browse/SENTRY-589 committed, we can use profile 
to control package hive-2 or not.
2. test-hive-v1 is enable as default, when do the pre-commit, how to test 
test-hive-v2?
Since we use profile to control the test, we need to create a new Jenkins Job 
for V2. I will request the permission at community, do you think it is possible 
to make Jenkins Passed for V1 currently.


- Dapeng


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


On 三月 24, 2016, 1:41 p.m., Dapeng Sun wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45260/
> ---
> 
> (Updated 三月 24, 2016, 1:41 p.m.)
> 
> 
> Review request for sentry and Colin Ma.
> 
> 
> Bugs: SENTRY-1159
> https://issues.apache.org/jira/browse/SENTRY-1159
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> Since Hive 2.0.0 depends on datanuclues 4+, we should decouple datanuclues 
> dependence on binding.
> 
> 
> Diffs
> -
> 
>   pom.xml d25c314 
>   sentry-binding/sentry-binding-hive/pom.xml fb5f214 
>   sentry-provider/sentry-provider-db/pom.xml bf4dfdc 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
>  c5c5ffb 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
>  94bd2a9 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestSentryRole.java
>  7b4d3b8 
>   sentry-tests/pom.xml 88a28bb 
>   sentry-tests/sentry-tests-hive-v2/pom.xml PRE-CREATION 
>   sentry-tests/sentry-tests-hive/pom.xml 472cce7 
>   sentry-tests/sentry-tests-kafka/pom.xml 54c7205 
>   sentry-tests/sentry-tests-solr/pom.xml e90ca06 
>   sentry-tests/sentry-tests-sqoop/pom.xml 3989ced 
> 
> Diff: https://reviews.apache.org/r/45260/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Dapeng Sun
> 
>

Re: Review Request 45260: SENTRY-1159: Decouple datanucleus dependences for hive-binding V1 and V2

[Dapeng Sun]

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

(Updated 三月 24, 2016, 1:41 p.m.)


Review request for sentry and Colin Ma.


Bugs: SENTRY-1159
https://issues.apache.org/jira/browse/SENTRY-1159


Repository: sentry


Description
---

Since Hive 2.0.0 depends on datanuclues 4+, we should decouple datanuclues 
dependence on binding.


Diffs (updated)
-

  pom.xml d25c314 
  sentry-binding/sentry-binding-hive/pom.xml fb5f214 
  sentry-provider/sentry-provider-db/pom.xml bf4dfdc 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
 c5c5ffb 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
 94bd2a9 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestSentryRole.java
 7b4d3b8 
  sentry-tests/pom.xml 88a28bb 
  sentry-tests/sentry-tests-hive-v2/pom.xml PRE-CREATION 
  sentry-tests/sentry-tests-hive/pom.xml 472cce7 
  sentry-tests/sentry-tests-kafka/pom.xml 54c7205 
  sentry-tests/sentry-tests-solr/pom.xml e90ca06 
  sentry-tests/sentry-tests-sqoop/pom.xml 3989ced 

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


Testing
---


Thanks,

Dapeng Sun

[discuss] ResourceAuthorizationProvider.hasAccess() performance

[nazgul33]

-- resending mail to dev@sentry.apache.org. I accidentaly sent mail to
d...@sentry.incubator.apache.org --

Hi, all

I'm Donghoon Han, a software developer.
I subscribed this mailing list to discuss performance problem of sentry.

My current job is to develop a type 3 JDBC driver for universal access to
Hive, Impala and Phoenix, which has it's own sentry enforcing logic.
We have 1000 users and 10k tables in a single hive metastore. (Huge!)

One problematic requirement is to implement JDBC api metadata.getTables()
which only lists tables connected user has access.

To evaluate all tables with 1000 public tables out of 10k tables, with 1000
public users,
current logic requires about 200 seconds.

I found that single hasAccess() call
- lists rules in groups where the user belongs : getPrivileges()
- iterate through the rules and test requested privilege.
- while iterating, text based rules are converted to Privilege object and
discarded.

I thought the performance problem lies in converting text rule to Privilege
object everytime.
So I wrote some code to experiment.

= first experiment =
1. get groups
2. list rules in groups and convert them to Privilege objects
3. put it in HashMap, where key is name of group.
4. next evaluation will use Privilege objects which are cached in HashMap.

this decreased full evaluation time from 200seconds to 10seconds.

= second experiment =
- most users are public users, with SELECT only.
- current EnumSet.allOf(DBModelAction.class) has ordering of INSERT,
SELECT, ALL
so I changed evaluation order to SELECT, INSERT, ALL.

10 seconds -> 5 seconds.

I believe pre-building Privilege object at loading time will greatly
enhance SENTRY's performance.
code snippet is available here. the code is "proof of idea" code.

http://pastebin.com/S7uQnTN7

If all agree that this could be a good idea, I think I can contribute some
code.

Thanks
 Donghoon Han

[discuss] ResourceAuthorizationProvider.hasAccess() performance

[nazgul33]

Hi, all

I'm Donghoon Han, a software developer.
I subscribed this mailing list to discuss performance problem of sentry.

My current job is to develop a type 3 JDBC driver for universal access to
Hive, Impala and Phoenix, which has it's own sentry enforcing logic.
We have 1000 users and 10k tables in a single hive metastore. (Huge!)

One problematic requirement is to implement JDBC api metadata.getTables()
which only lists tables connected user has access.

To evaluate all tables with 1000 public tables out of 10k tables, with 1000
public users,
current logic requires about 200 seconds.

I found that single hasAccess() call
- lists rules in groups where the user belongs : getPrivileges()
- iterate through the rules and test requested privilege.
- while iterating, text based rules are converted to Privilege object and
discarded.

I thought the performance problem lies in converting text rule to Privilege
object everytime.
So I wrote some code to experiment.

= first experiment =
1. get groups
2. list rules in groups and convert them to Privilege objects
3. put it in HashMap, where key is name of group.
4. next evaluation will use Privilege objects which are cached in HashMap.

this decreased full evaluation time from 200seconds to 10seconds.

= second experiment =
- most users are public users, with SELECT only.
- current EnumSet.allOf(DBModelAction.class) has ordering of INSERT,
SELECT, ALL
so I changed evaluation order to SELECT, INSERT, ALL.

10 seconds -> 5 seconds.

I believe pre-building Privilege object at loading time will greatly
enhance SENTRY's performance.
code snippet is available here. the code is "proof of idea" code.

http://pastebin.com/S7uQnTN7

If all agree that this could be a good idea, I think I can contribute some
code.

Thanks
 Donghoon Han

Sentry-jdk-1.7 - Build # 614 - Still Unstable

[Apache Jenkins Server]

The Apache Jenkins build system has built Sentry-jdk-1.7 (build #614)

Status: Still Unstable

Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/614/ to 
view the results.

Re: Review Request 45260: SENTRY-1159: Decouple datanucleus dependences for hive-binding V1 and V2

[Colin Ma]

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



Thanks for the patch, I have 2 comments:
1. Different versions for datanucleus are included for hive-v1 and hive-v2. 
When package Sentry, the lib will include all these jars, how to avoid the 
conflict?
2. test-hive-v1 is enable as default, when do the pre-commit, how to test 
test-hive-v2?

- Colin Ma


On March 24, 2016, 1:25 a.m., Dapeng Sun wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45260/
> ---
> 
> (Updated March 24, 2016, 1:25 a.m.)
> 
> 
> Review request for sentry and Colin Ma.
> 
> 
> Bugs: SENTRY-1159
> https://issues.apache.org/jira/browse/SENTRY-1159
> 
> 
> Repository: sentry
> 
> 
> Description
> ---
> 
> Since Hive 2.0.0 depends on datanuclues 4+, we should decouple datanuclues 
> dependence on binding.
> 
> 
> Diffs
> -
> 
>   pom.xml d25c314 
>   sentry-binding/sentry-binding-hive/pom.xml fb5f214 
>   sentry-provider/sentry-provider-db/pom.xml bf4dfdc 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
>  c5c5ffb 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
>  94bd2a9 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestSentryRole.java
>  7b4d3b8 
>   sentry-tests/pom.xml 88a28bb 
>   sentry-tests/sentry-tests-hive-v2/pom.xml PRE-CREATION 
>   sentry-tests/sentry-tests-hive/pom.xml 472cce7 
> 
> Diff: https://reviews.apache.org/r/45260/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Dapeng Sun
> 
>

Review Request 45262: SENTRY-1137: Update hive dependence to 2.0.0

[Dapeng Sun]

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

Review request for sentry and Colin Ma.


Bugs: SENTRY-1130
https://issues.apache.org/jira/browse/SENTRY-1130


Repository: sentry


Description
---

Updated hive-binding-v2 for Hive 2.0.0


Diffs
-

  pom.xml d25c314 
  sentry-binding/sentry-binding-hive-v2/pom.xml ef6048c 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingHookV2.java
 67cf266 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzBindingSessionHookV2.java
 3fbb626 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/HiveAuthzPrivilegesMapV2.java
 PRE-CREATION 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/SentryAuthorizerFactory.java
 4a5cbcf 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java
 70e0720 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/AuthorizingObjectStoreV2.java
 726f5ad 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/MetastoreAuthzBindingV2.java
 d937491 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/metastore/SentryMetastorePostEventListenerV2.java
 013d016 
  
sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/util/SimpleSemanticAnalyzer.java
 b50bbf4 

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


Testing
---


Thanks,

Dapeng Sun

Re: Sentry Graduation Blog

[Hao Hao]

Thanks a lot Sravya for the suggestions! Will add the HDFS sync feature and
I have created the Jira for creating a apache blog account.

Best,
Hao

On Wed, Mar 23, 2016 at 11:37 AM, Sravya Tirukkovalur 
wrote:

> Thanks a lot for putting this together Hao! A graduation blog post is a
> great idea!
>
> One comment: Do we want to a line or two about HDFS sync feature and how it
> allows setting rules for hive databases and tables in one place, rather
> than manually keeping the Hive permissions and HDFS permissions in sync?
>
> Also, do you have a apache blog account? If not, please file an infra
> ticket. See INFRA-6911 for example.
>
>
> On Tue, Mar 22, 2016 at 11:53 AM, Hao Hao  wrote:
>
> > Hi All,
> >
> > I drafted a gradation blog post for Sentry and attached here.  Please
> feel
> > free to give recommendations and comments. Thanks a lot!
> >
> > Best,
> > Hao
> >
> > Sentry Graduation Blog
> >
> > We are very excited to announce that Apache Sentry
> >  has graduated out of Incubator and is now an
> > Apache Top Level Project! Sentry, which provides centralized fine-grained
> > access control on metadata and data stored on Apache Hadoop cluster, is
> > introduced as an Apache Incubator project back in August 2013. In the
> past
> > two and half years, the development community grew significantly to a
> large
> > number of contributors from various organizations. Upon graduation, there
> > were more than 50 contributors, 31 of whom had become committers.
> >
> > What’s Sentry
> >
> > While Hadoop has strong security at the filesystem level, it lacked the
> > granular support needed to adequately secure access to data by users and
> BI
> > applications. This problem forces users to make a choice: either leave
> data
> > unprotected or lock out users entirely. Most of the time, the preferred
> > choice is the latter, severely inhibiting access to data in Hadoop.
> Sentry
> > provides the ability to enforce role-based access control to data and/or
> > privileges on data for authenticated users in a fine-grained manner. For
> > example, Sentry allows access control at the server, database, table,
> view
> > and even column scope at different privilege levels including select,
> > insert, and all for Apache Hive and Apache Impala.
> >
> >
> > What’s new
> >
> > During incubation, Sentry had six releases and has continued to grow on
> > providing unified authorization policy management across different Hadoop
> > components.  Some of them including:
> >
> >-
> >
> >most recent integration support with Apache Kafka, Apache Solr and
> >Apache Sqoop;
> >-
> >
> >audit log support for data governance purpose;
> >-
> >
> >improve the service stability with Sentry High Availability (HA);
> >-
> >
> >provides import/export tool for replicating the entire permissions on
> a
> >Sentry database.
> >
> > Future Work
> >
> > Graduation is a terrific milestone, but only the beginning for Sentry. We
> > are looking forward to continuing to help grow the Sentry community and
> > fostering a strong ecosystem around the project.
> >
> > We are targeting at strength Sentry core with finer granularity
> > authorization model, as well as enable easier integration process with
> > other components to make Sentry highly adaptable. Meanwhile we are aiming
> > at supportability enhancements such as bettering Sentry Web UI.
> >
> > How to Get Involved
> >
> > The Sentry community now includes new core committers, an active
> developer
> > mailing list where future releases and patches are discussed, and
> > increasing interest in running additional frameworks on Sentry. We
> strongly
> > encourage new people join Sentry and contribute
> > 
> > through jumping on the discussions on the mailing list, filing bugs
> through
> > Jira, reviewing other's’ code or even providing new patches.
> >
>
>
>
> --
> Sravya Tirukkovalur
>

Re: [Discuss] Apache Sentry 1.7.0 Release

[Hao Hao]

Hi Sravya,

Will send out a proposal for branching date later today. :) Thanks!

Best,
Hao

On Wed, Mar 23, 2016 at 11:52 AM, Sravya Tirukkovalur 
wrote:

> Hi Hao,
>
> Just wanted to check, when are we branching?
>
>
> On Tue, Mar 1, 2016 at 11:20 PM, Hao Hao  wrote:
>
> > Thanks Sravya!
> >
> > Best,
> > Hao
> >
> > On Tue, Mar 1, 2016 at 11:10 PM, Sravya Tirukkovalur <
> sra...@cloudera.com>
> > wrote:
> >
> > > Perfect! See this for the process Hao:
> > > https://cwiki.apache.org/confluence/display/SENTRY/How+to+Release
> > >
> > >
> > > On Tue, Mar 1, 2016 at 11:05 PM, Hao Hao  wrote:
> > >
> > > > I can start to prepare for the 1.7.0 Release. Will follow up shortly.
> > > > Thanks!
> > > >
> > > > Best,
> > > > Hao
> > > >
> > > > On Tue, Mar 1, 2016 at 10:58 PM, Sravya Tirukkovalur <
> > > sra...@cloudera.com>
> > > > wrote:
> > > >
> > > > > +1, seems like a good time to make a release. From other threads I
> > see
> > > > that
> > > > > there are also some users waiting for 1.7.0 release for generic
> model
> > > > > improvements
> > > > >
> > > > > Any volunteers to drive the release?
> > > > >
> > > > > On Sun, Feb 21, 2016 at 4:55 PM, Ma, Junjie 
> > > wrote:
> > > > >
> > > > > >
> > > > > > Agree to launch a new release, but for the Hive_plugin_v2, the
> e2e
> > > test
> > > > > is
> > > > > > not available now. I think it shouldn't be include in this
> release.
> > > > > >
> > > > > > Best regards,
> > > > > >
> > > > > > Colin Ma(Ma Jun Jie)
> > > > > >
> > > > > > -Original Message-
> > > > > > From: Hao Hao [mailto:hao@cloudera.com]
> > > > > > Sent: Saturday, February 20, 2016 6:20 AM
> > > > > > To: d...@sentry.incubator.apache.org
> > > > > > Subject: [Discuss] Apache Sentry 1.7.0 Release
> > > > > >
> > > > > > Hi All,
> > > > > >
> > > > > > Should we launch a new release since 1.6.0 Release is four month
> > > back?
> > > > We
> > > > > > have new features such as Kafak-Sentry integration and
> > > Hive_plugin_v2.
> > > > > And
> > > > > > a bunch of improvements including the performance improvement for
> > > > column
> > > > > > level privileges and enabling of PMD plugin for ensuring code
> > > > quailities.
> > > > > > Also a lot of fix including the fix for sentry generic model.
> > > > Probabaly
> > > > > > we should start to do 1.7.0 release after the planned new
> features
> > > are
> > > > > done?
> > > > > > Any thoughts? Thanks!
> > > > > >
> > > > > > Best,
> > > > > > Hao
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Sravya Tirukkovalur
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Sravya Tirukkovalur
> > >
> >
>
>
>
> --
> Sravya Tirukkovalur
>

Re: Sentry Graduation Blog

[Sravya Tirukkovalur]

Thanks a lot for putting this together Hao! A graduation blog post is a
great idea!

One comment: Do we want to a line or two about HDFS sync feature and how it
allows setting rules for hive databases and tables in one place, rather
than manually keeping the Hive permissions and HDFS permissions in sync?

Also, do you have a apache blog account? If not, please file an infra
ticket. See INFRA-6911 for example.


On Tue, Mar 22, 2016 at 11:53 AM, Hao Hao  wrote:

> Hi All,
>
> I drafted a gradation blog post for Sentry and attached here.  Please feel
> free to give recommendations and comments. Thanks a lot!
>
> Best,
> Hao
>
> Sentry Graduation Blog
>
> We are very excited to announce that Apache Sentry
>  has graduated out of Incubator and is now an
> Apache Top Level Project! Sentry, which provides centralized fine-grained
> access control on metadata and data stored on Apache Hadoop cluster, is
> introduced as an Apache Incubator project back in August 2013. In the past
> two and half years, the development community grew significantly to a large
> number of contributors from various organizations. Upon graduation, there
> were more than 50 contributors, 31 of whom had become committers.
>
> What’s Sentry
>
> While Hadoop has strong security at the filesystem level, it lacked the
> granular support needed to adequately secure access to data by users and BI
> applications. This problem forces users to make a choice: either leave data
> unprotected or lock out users entirely. Most of the time, the preferred
> choice is the latter, severely inhibiting access to data in Hadoop. Sentry
> provides the ability to enforce role-based access control to data and/or
> privileges on data for authenticated users in a fine-grained manner. For
> example, Sentry allows access control at the server, database, table, view
> and even column scope at different privilege levels including select,
> insert, and all for Apache Hive and Apache Impala.
>
>
> What’s new
>
> During incubation, Sentry had six releases and has continued to grow on
> providing unified authorization policy management across different Hadoop
> components.  Some of them including:
>
>-
>
>most recent integration support with Apache Kafka, Apache Solr and
>Apache Sqoop;
>-
>
>audit log support for data governance purpose;
>-
>
>improve the service stability with Sentry High Availability (HA);
>-
>
>provides import/export tool for replicating the entire permissions on a
>Sentry database.
>
> Future Work
>
> Graduation is a terrific milestone, but only the beginning for Sentry. We
> are looking forward to continuing to help grow the Sentry community and
> fostering a strong ecosystem around the project.
>
> We are targeting at strength Sentry core with finer granularity
> authorization model, as well as enable easier integration process with
> other components to make Sentry highly adaptable. Meanwhile we are aiming
> at supportability enhancements such as bettering Sentry Web UI.
>
> How to Get Involved
>
> The Sentry community now includes new core committers, an active developer
> mailing list where future releases and patches are discussed, and
> increasing interest in running additional frameworks on Sentry. We strongly
> encourage new people join Sentry and contribute
> 
> through jumping on the discussions on the mailing list, filing bugs through
> Jira, reviewing other's’ code or even providing new patches.
>



-- 
Sravya Tirukkovalur

Review Request 45221: Add a test run to exclude slow and not thread safe tests. It could be used for pre-commit jobs

[Anne Yu]

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

Review request for sentry.


Bugs: SENTRY-1108
https://issues.apache.org/jira/browse/SENTRY-1108


Repository: sentry


Description
---

The way to run it: mvn test -P skipSlowAndNotThreadSafeTests; So it won't 
affect default mvn test;
Most excluded slow tests are from sentry-tests/e2e suites and a few are those 
can't be run under concurrent mode.


Diffs
-

  pom.xml d25c314e0a4c92ea03f22c01c07e0b0beba2bbea 

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


Testing
---

Test the running time on an ec2 cluster (large). Run mvn clean install first 
then mvn test.

Full suite: 2039 tests, 92m58.727s;
skipSlowAndNotThreadSafeTests run: 1027 tests, 6m47.195s;


Thanks,

Anne Yu