Re: DBI dependency in Sedona R

2022-09-16 Thread Felix Cheung 
>From what I can see, it’s only directly used in two places

- tests
- sql variant
https://github.com/apache/incubator-sedona/blob/65080422b6256a0f12cc74347ac2ed531d26a4f6/R/R/dependencies.R#L135

Don’t know enough about dbplyr to tell if that functionality can be turned
off (say if sql is not used)


On Wed, Sep 14, 2022 at 12:05 AM Jia Yu  wrote:

> This is a question raised by the incubator community regarding graduating
> Sedona to a TLP.
>
> On Tue, Sep 13, 2022 at 11:57 PM Jia Yu  wrote:
>
> > Hi Yitao,
> >
> > The DBI dependency in Sedona R (
> > https://github.com/apache/incubator-sedona/blob/master/R/DESCRIPTION) is
> > under LGPL 2.1 license, which may have compatibility issues with ASF. Is
> it
> > mandatory? If not, can we remove it?
> >
> > Thanks,
> > Jia
> >
>

Re: DBI dependency in Sedona R

2022-09-14 Thread Jia Yu 
This is a question raised by the incubator community regarding graduating
Sedona to a TLP.

On Tue, Sep 13, 2022 at 11:57 PM Jia Yu  wrote:

> Hi Yitao,
>
> The DBI dependency in Sedona R (
> https://github.com/apache/incubator-sedona/blob/master/R/DESCRIPTION) is
> under LGPL 2.1 license, which may have compatibility issues with ASF. Is it
> mandatory? If not, can we remove it?
>
> Thanks,
> Jia
>

DBI dependency in Sedona R

2022-09-14 Thread Jia Yu 
Hi Yitao,

The DBI dependency in Sedona R (
https://github.com/apache/incubator-sedona/blob/master/R/DESCRIPTION) is
under LGPL 2.1 license, which may have compatibility issues with ASF. Is it
mandatory? If not, can we remove it?

Thanks,
Jia