You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wayang.apache.org by "kbeedkar (via GitHub)" <gi...@apache.org> on 2023/04/15 05:27:47 UTC
[GitHub] [incubator-wayang] kbeedkar opened a new pull request, #284: Initial version of the SQL API
kbeedkar opened a new pull request, #284:
URL: https://github.com/apache/incubator-wayang/pull/284
- This initial version of the SQL API supports simple Select-Project Queries over Databases and Java Platform.
- SQL support is based on Apache Calcite
--
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: commits-unsubscribe@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [GitHub] [incubator-wayang] kbeedkar opened a new pull request, #284: Initial version of the SQL API
Posted by Kaustubh Beedkar <kb...@gmail.com>.
Hi folks!
I created a PR, which contributes to the first basic version of the SQL
API. Kindly review and let me know your thoughts.
Best,
Kaustubh
On Sat, Apr 15, 2023 at 10:57 AM kbeedkar (via GitHub) <gi...@apache.org>
wrote:
>
> kbeedkar opened a new pull request, #284:
> URL: https://github.com/apache/incubator-wayang/pull/284
>
> - This initial version of the SQL API supports simple Select-Project
> Queries over Databases and Java Platform.
> - SQL support is based on Apache Calcite
>
>
>
>
> --
> 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: commits-unsubscribe@wayang.apache.org
>
> For queries about this service, please contact Infrastructure at:
> users@infra.apache.org
>
>
[GitHub] [incubator-wayang] 2pk03 merged pull request #284: Initial version of the SQL API
Posted by "2pk03 (via GitHub)" <gi...@apache.org>.
2pk03 merged PR #284:
URL: https://github.com/apache/incubator-wayang/pull/284
--
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: commits-unsubscribe@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] 2pk03 commented on pull request #284: Initial version of the SQL API
Posted by "2pk03 (via GitHub)" <gi...@apache.org>.
2pk03 commented on PR #284:
URL: https://github.com/apache/incubator-wayang/pull/284#issuecomment-1509626458
Hey @kbeedkar ,
awesome contribution, thank you! SQL implementation with Apache Calcite is upmost important. Please check the license issues, and re-submit your PR. Codewise everything is fine, I could compile on Mac M1.
--
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: commits-unsubscribe@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] kbeedkar commented on pull request #284: Initial version of the SQL API
Posted by "kbeedkar (via GitHub)" <gi...@apache.org>.
kbeedkar commented on PR #284:
URL: https://github.com/apache/incubator-wayang/pull/284#issuecomment-1509645978
Hi @Alex, Fixed the license issues. Pls. check again.
--
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: commits-unsubscribe@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] 2pk03 commented on pull request #284: Initial version of the SQL API
Posted by "2pk03 (via GitHub)" <gi...@apache.org>.
2pk03 commented on PR #284:
URL: https://github.com/apache/incubator-wayang/pull/284#issuecomment-1509667283
@kbeedkar, build failed here, can you please check?
https://github.com/apache/incubator-wayang/actions/runs/4706923589/jobs/8348520563?pr=284#step:7:1981
--
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: commits-unsubscribe@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] 2pk03 commented on pull request #284: Initial version of the SQL API
Posted by "2pk03 (via GitHub)" <gi...@apache.org>.
2pk03 commented on PR #284:
URL: https://github.com/apache/incubator-wayang/pull/284#issuecomment-1509661986
Strict licence check:
```
Warning: There are 2 forbidden licenses used:
Warning: License: 'iCal4j - License' used by 1 dependencies:
-ical4j (org.mnode.ical4j:ical4j:1.0.2 - [http://ical4j.sourceforge.net)](http://ical4j.sourceforge.net%29/)
Warning: License: 'Public Domain' used by 1 dependencies:
-Backport of JSR 166 (backport-util-concurrent:backport-util-concurrent:3.1 - http://backport-jsr166.sourceforge.net/)
```
This should not be a problem, since Public Domain is a permissive license:
[Public Domain](https://creativecommons.org/licenses/publicdomain/). I'll check why the build check failed.
--
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: commits-unsubscribe@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] kbeedkar commented on pull request #284: Initial version of the SQL API
Posted by "kbeedkar (via GitHub)" <gi...@apache.org>.
kbeedkar commented on PR #284:
URL: https://github.com/apache/incubator-wayang/pull/284#issuecomment-1510095981
I could now build successfully locally on my machine. Hopefully, there are no more license issues now.
--
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: commits-unsubscribe@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org