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

Improve NON_KEYWORDS and other changes #3532

Merged
merged 13 commits into from Jun 7, 2022
Merged

Conversation

katzyn
Copy link
Contributor

@katzyn katzyn commented Jun 5, 2022

  1. NON_KEYWORDS setting now works in more cases, for example, ROW now can be used as a column name with NON_KEYWORDS=ROW. Closes "ROW" cannot be set as a non keyword in 2.x #3390.
  2. FILTER is removed from the list of context-sensitive keywords, because now only FILTER (WHERE has special meaning.
  3. Arguments of methods in SpatialDataType are changed from Object to Spatial to reduce number of casts. Spatial is an interface from MVStore, so it can be used here without introduction of unwanted dependencies. Object was needed a long time ago to avoid usage of SpatialKey from H2.
  4. DbObject.getCreateSQLForCopy() now has default implementation that throws an exception to significantly reduce number of its implementations, most of them also need to throw an exception.
  5. Database.getTempTableName() now truncates too long names of identifiers.
  6. Tables with custom table engines now can be only created by an user with ADMIN privileges like any other DDL operations with user-provided classes.

@katzyn katzyn merged commit 28eec32 into h2database:master Jun 7, 2022
@katzyn katzyn deleted the misc branch June 7, 2022 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"ROW" cannot be set as a non keyword in 2.x
1 participant