Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve codecs inside Macro #83

Open
tabdulradi opened this issue Sep 4, 2016 · 1 comment
Open

Resolve codecs inside Macro #83

tabdulradi opened this issue Sep 4, 2016 · 1 comment

Comments

@tabdulradi
Copy link
Member

Instead of emitting code that accepts implicit TroyCodec instances,
resolve them inside the macro first, aborting if the codec is not found.

This allows us to

  1. Customise error message, by hiding internal CassandraDataType.Type classes, and show the types as lowercase "text", "map <int, text>", as it would appear inside CQLsh
  2. Validate queries against multiple versions of the Schema, for every version, we can make sure Codecs exists
@tabdulradi
Copy link
Member Author

tabdulradi commented Sep 4, 2016

Based on #84, the story has been removed from the milestone, and now have lower priority. Since Troy won't support parsing a query the involves columns that changed their types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant