-
Notifications
You must be signed in to change notification settings - Fork 267
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
Add cnosdb expect error and modify db schema #735
base: main
Are you sure you want to change the base?
Conversation
add expect error and modify db schema
add expect error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! Could you please have a look at my two minor comments?
@@ -69,6 +69,8 @@ private CnosDBExpectedError() { | |||
ERRORS.add( | |||
"Coercion from [Utf8, Float64, Utf8] to the signature OneOf([Exact([Utf8, Int64]), Exact([LargeUtf8, Int64]), Exact([Utf8, Int64, Utf8]), Exact([LargeUtf8, Int64, Utf8]), Exact([Utf8, Int64, LargeUtf8]), Exact([LargeUtf8, Int64, LargeUtf8])]) failed."); | |||
ERRORS.add("Coercion from"); | |||
ERRORS.add("Build left right indices error"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For new expected errors, I'd suggest adding them on a finer-grained basis (e.g., having errors for expression evaluations, specific kinds of statements, etc).
@@ -69,9 +69,10 @@ public CnosDBConnection createDatabase(CnosDBGlobalState globalState) throws Exc | |||
CnosDBClient client = new CnosDBClient(host, port, username, password, databaseName); | |||
CnosDBConnection connection = new CnosDBConnection(client); | |||
client.execute("DROP DATABASE IF EXISTS " + databaseName); | |||
String createdb = "CREATE DATABASE " + databaseName + " WITH TTL '1000000' REPLICA 2"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does WITH TTL '1000000'
mean? Do you think it might be useful to add a short comment why we override the default configuration?
Any updates on this? It would be great if we could merge the PR (which is now out-of-date with the base branch, so a merge would be necessary)! |
Add cnosdb expect error and modify db schema