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
> >
>


[GitHub] [incubator-sedona] jiayuasu merged pull request #688: [SEDONA-168] Add ST_Normalize to Apache Sedona

2022-09-16 Thread GitBox


jiayuasu merged PR #688:
URL: https://github.com/apache/incubator-sedona/pull/688


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@sedona.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [incubator-sedona] jiayuasu merged pull request #690: [DOCS] remove DBI as a R dependency

2022-09-16 Thread GitBox


jiayuasu merged PR #690:
URL: https://github.com/apache/incubator-sedona/pull/690


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@sedona.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org