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

update Tibero new gis view/table spec #28

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TiberoClient
Copy link

@TiberoClient TiberoClient commented Jan 23, 2024

With the update of Tibero version, the specifications of views and tables related to GIS have been changed.
This commit reflects those changes.
We also tried to maintain compatibility with previous versions.
Here are some key changes:

  • SYSGIS.GEOMETRY_COLUMNS_BASE table, which used to store GIS metadata, has been removed.
    Therefore, users no longer need to execute DML directly on this table or run procedures.
    ALL(USER)_GEOMETRY_COLUMNS view will be updated automatically.
  • The column specifications of the ALL(USER)_GEOMETRY_COLUMNS view have been changed.
  • Views/tables related to units have been added.

This is a link for checking some new specifications
<Tibero Spatial 참조 안내서 in Tibero Online Manual>

Here are the key updates made in TiberoDialect.java:

  • The names of GIS-related views/tables have been grouped into the GisTable enum.
    This is to facilitate the management of additional GIS-related views/tables that may be added in the future.
  • To maintain compatibility between previous and current specifications, an isOldDBVersion flag has been introduced to create a branch.
    This flag is determined based on the existence of the SYSGIS.GEOMETRY_COLUMNS_BASE table and the F_GEOMETRY_TYPE column in the ALL_GEOMETRY_COLUMNS view.
  • The logic to execute DML on SYSGIS.GEOMETRY_COLUMNS_BASE in postCreateTable(), postDropTable() has been made to proceed only in previous versions.
  • The query for querying UDT(user-defined type) has been changed.

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.

1 participant