---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66792/#review202073
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66787/#review202072
---
Ship it!
Thanks, the change looks better.
FYI, keeping the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66787/
---
(Updated April 27, 2018, 4:47 p.m.)
Review request for sentry.
Changes
> On April 27, 2018, 3:51 p.m., kalyan kumar kalvagadda wrote:
> > Lina, This index might not be needed for every database. we need it for
> > Oracle database for sure.
> >
> > For example, mysql creates an index automaticallym when a cloumn is added
> > as a foreign key. Adding a duplicate
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66764/#review202069
---
Lina, This index might not be needed for every database. we need
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66748/
---
(Updated April 27, 2018, 3:47 p.m.)
Review request for sentry.
Repository:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66764/#review202067
---
Ship it!
Ship It!
- Sergio Pena
On April 27, 2018, 3:13
> On April 23, 2018, 9:59 p.m., kalyan kumar kalvagadda wrote:
> > You need to update the new version in SentryStoreSchemaInfo.java.Please
> > make this change ans run the upgrade/install tests to make sure thay are
> > fine.
it is done
- Na
> On April 25, 2018, 8:21 p.m., Sergio Pena wrote:
> > Lina, would it be better to create another patch to fix the 2.1.0 versions?
> > So we can have this patch to add the index only instead of several new
> > files, what do you think?
>
> Na Li wrote:
> Sergio, I don't want to take the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66764/
---
(Updated April 27, 2018, 3:13 p.m.)
Review request for sentry, Alexander
10 matches
Mail list logo
