lbadd icon indicating copy to clipboard operation
lbadd copied to clipboard

parser: support SQL comments

Open tsatke opened this issue 3 years ago • 13 comments

Extend the parser so that he supports comments as specified here (https://www.sqlite.org/lang_comment.html).

tsatke avatar Jul 10 '20 20:07 tsatke

Hello, I would like to grab this issue and work on it if no one is working on it.

Shivam010 avatar Sep 07 '20 07:09 Shivam010

Hi, please move to xqueries/xdb and find that issue. We no longer work in this repo

xqueries/xdb#1

tsatke avatar Sep 07 '20 07:09 tsatke

You're more than welcome to pick up this issue here if you'd like.

Tim has made a fork and has moved to that, however you're welcome to contribute to whichever you would like.

I'd be happy to review a PR if you open it.

tomarrell avatar Sep 07 '20 07:09 tomarrell

Ok, but is there any particular reason for maintaining two different project works? I am kinda confused with the parting. Can I know the goal set or the differences between both the projects if possible, @tomarrell @tsatke?

Shivam010 avatar Sep 07 '20 07:09 Shivam010

I would really like to work on building a database. I like the project.

Shivam010 avatar Sep 07 '20 07:09 Shivam010

@tomarrell is inactive since about febuary, and I'm not an admin in this repo, so I can't archive it or anything

tsatke avatar Sep 07 '20 07:09 tsatke

the goal set of the new project is just a different roadmap really

tsatke avatar Sep 07 '20 07:09 tsatke

Let's take this discussion to xqueries/xdb#1

tsatke avatar Sep 07 '20 07:09 tsatke

Ok, if it's about inactivity

Shivam010 avatar Sep 07 '20 07:09 Shivam010

Sure, I have certainly been less than active, that is true. However, I would not consider this project abandoned by any means. I would consider it more of a long running project.

The goal of this project when I first embarked on it was to build something that was ultimately as simple as possible in order to act as a good reference for people who want to learn the inner workings of a distributed database.

If you share the same goals, then you're more than welcome to contribute here if you would like.

Otherwise, xqueries/xdb looks to be a good option if you would like to take it past this above goal, as it appears they're looking to build something a little more tuned for production use.

tomarrell avatar Sep 07 '20 07:09 tomarrell

Yes, I would fully agree with that description! Feel free to choose either one, you're welcome in both projects!

tsatke avatar Sep 07 '20 08:09 tsatke

Right now I would like to choose xdb. No hard feelings.

Shivam010 avatar Sep 07 '20 08:09 Shivam010

No worries at all. Best of luck!

tomarrell avatar Sep 07 '20 08:09 tomarrell