Skip to content

added relevant yaml files and testcases for basic check for oracle#37816

Open
manasagar wants to merge 3 commits intoapache:masterfrom
manasagar:37766
Open

added relevant yaml files and testcases for basic check for oracle#37816
manasagar wants to merge 3 commits intoapache:masterfrom
manasagar:37766

Conversation

@manasagar
Copy link
Contributor

Fixes #37766.

Changes proposed in this pull request:

  • Added relevant yaml files which were created by querying an empty oracle db for all static dictionary tables and converting them to yaml files
  • db docker image gvenzl/oracle-xe:18
  • reference https://docs.oracle.com/database/timesten-18.1/TTSYS/systemtables.htm)
  • query -
    SELECT view_name
    FROM all_views
    WHERE owner = 'SYS'
    AND (view_name LIKE 'ALL\%' ESCAPE '\'
    OR view_name LIKE 'DBA\
    %' ESCAPE '\'
    OR view_name LIKE 'USER\_%' ESCAPE '\')
    AND view_name NOT LIKE '%\AW\%' ESCAPE '\'
    ORDER BY view_name

Before committing this PR, I'm sure that I have checked the following options:

  • My code follows the code of conduct of this project.
  • I have self-reviewed the commit code.
  • I have (or in comment I request) added corresponding labels for the pull request.
  • I have passed maven check locally : ./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e.
  • I have made corresponding changes to the documentation.
  • I have added corresponding unit tests for my changes.
  • I have updated the Release Notes of the current development version. For more details, see Update Release Note

@manasagar
Copy link
Contributor Author

@terrymanu PTAL

Copy link
Member

@terrymanu terrymanu left a comment

Choose a reason for hiding this comment

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

No one can review this many changes.
And there are so many attributes may impact performance; it's necessary to test first or consider alternative solutions.

@manasagar
Copy link
Contributor Author

manasagar commented Jan 25, 2026

No one can review this many changes. And there are so many attributes may impact performance; it's necessary to test first or consider alternative solutions.

I understand than the solution for this problem will require that sharding sphere itself recognises these tables in a db instead off us defining it any advice in this direction would be appreciated because if we only add partial number of tables on our own it will give error when it cant find a different table or please propose what type of test is required for the current implementation

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.

ShardingSphere-JDBC 5.5.2 encounters the error under Oracle: Table or view 'all_sequences' does not exist

2 participants