FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: Foundation for DQE Oracle Implementation by AmrRiyad · Pull Request #1196 · sqlancer/sqlancer · GitHub

feat: Foundation for DQE Oracle Implementation - #1196

Open
AmrRiyad wants to merge 1 commit into
sqlancer:mainfrom
AmrRiyad:introduce-dqe-base
Open

feat: Foundation for DQE Oracle Implementation#1196
AmrRiyad wants to merge 1 commit into
sqlancer:mainfrom
AmrRiyad:introduce-dqe-base

Conversation

AmrRiyad commented Mar 31, 2025
edited
Loading

Copy link
Copy Markdown
Contributor

Description

This PR lays the groundwork for implementing the DQE oracle across multiple DBMS by introducing a new abstract class in DQEBase.java. The class serves as the base for handling auxiliary columns and error management, setting the stage for DBMS-specific extensions in the future.

Proposed Changes

  • Abstract Class Implementation:

    • Introduce DQEBase as the core abstract class for the DQE oracle.
  • Auxiliary Column Management:

    • Define the abstract method addAuxiliaryColumns(), which concrete implementations must override.

    • Provide a helper method, dropAuxiliaryColumns(), to consistently remove auxiliary columns (rowId and updated) from tables.

  • Errors Comparison:

    • Implement a new SQLQueryError class that provides comprehensive comparisons of error details—such as levels, codes, and messages—from multiple executed queries.

Impact

  • Modularity and Extensibility:
    • Establishes a clear foundation that allows for multiple DBMS-specific implementations of the DQE oracle.

AmrRiyad changed the title feat: introduce DQEBase and SQLQueryError feat: Foundation for DQE Oracle Implementation Mar 31, 2025

Copy link
Copy Markdown
Contributor

The modifications look good to me.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL