Skip to content
This repository was archived by the owner on Apr 22, 2020. It is now read-only.

Feature/cql#475

Open
peter-janssen wants to merge 2 commits intogooglearchive:masterfrom
peter-janssen:feature/cql
Open

Feature/cql#475
peter-janssen wants to merge 2 commits intogooglearchive:masterfrom
peter-janssen:feature/cql

Conversation

@peter-janssen
Copy link
Copy Markdown

As per #451 the required changes to add cql extension. I basically made a copy of the sql extension and replaced the keywords with those from cassandra. The tests are copied over also but with a usernames table instead of users because the latter is a reserved keyword in cql.

@googlebot
Copy link
Copy Markdown

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@peter-janssen
Copy link
Copy Markdown
Author

@googlebot I signed it!

@googlebot
Copy link
Copy Markdown

CLAs look good, thanks!

@peter-janssen peter-janssen force-pushed the feature/cql branch 4 times, most recently from d865650 to 814ca86 Compare February 12, 2017 23:06
Comment thread src/lang-cql.js Outdated
@@ -0,0 +1,55 @@
/**
* @license
* Copyright (C) 2008 Google Inc.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please put yourself down as the copyright holder. The Apache2 verbiage below then is your grant of license.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants