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

QP-3639 QD-1167 Fix MySQL invisible columns #57

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Web-Engine
Copy link
Contributor

@Web-Engine Web-Engine commented May 11, 2023

Summary

MySQL 8.0 및 MariaDB 10.3에서 추가된 invisible 컬럼에 대해 DML 동작을 지원합니다.

뭔가 굉장히 사이드 이펙트가 큰 변경 같아서 마음에 안들고 불안합니다.

Tests

  • MySQL 기존 테스트 케이스는 모두 통과하였습니다.
  • 새로 추가한 간단한 invisible 컬럼에 대한 테스트 케이스를 통과하였습니다.

Changes

  • 기존 테스트 중 ScriptHistory 순서에 대한 문제가 있어서, 해당 부분만 변경하였습니다.
  • invisible 컬럼에 대한 간단한 테스트 케이스가 추가되었습니다.
  • invisible 컬럼의 Before 데이터를 조회하기 위해 SELECT *, invisible_column1, invisible_column2, ... FROM ... 형태로 조회하도록 변경하였습니다.
  • 위에 조회 방식의 변경에 따라 SourceOrder를 변경하였습니다.
  • Table Lookup 시 Invisible 컬럼을 포함한 결과를 반환하지만, MySqlExplicitReferenceTable의 경우 Visible 컬럼만 조회되므로 Visible 컬럼만 나타나도록 예외처리 했습니다.

Issue

QP-3639
QD-1167

@Web-Engine Web-Engine changed the title QD-1167 Fix MySQL invisible columns QP-3639 QD-1167 Fix MySQL invisible columns May 11, 2023
@Web-Engine Web-Engine added the bug Something isn't working label Apr 22, 2024
@Web-Engine Web-Engine added this to the monitoring milestone Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

1 participant