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

Fix metadata change column error #2234

Merged
merged 3 commits into from
Jun 13, 2019
Merged

Conversation

brandon-wonjune
Copy link
Contributor

Description

메타데이터 컬럼의 타입 변경시 TIMSTAMP 타입이 아닌 경우 format이 남아있던 현상 수정
메타데이터 컬럼의 role이 TIMESTAMP인 경우 format을 변경할 수 있는 아이콘이 나오는 현상 수정 (데이터소스와 동일)
disable 된 컬럼 위에 컬럼 타입 목록이 열려있는경우 해당 위치의 타입을 클릭할 수 없던 현상 수정

Related Issue :

통합테스트

How Has This Been Tested?

  1. 메타데이터 > 컬럼 Dictionary를 type을 TIMESTAMP가 아닌 타입으로 생성 (format이 없는타입)
  2. format 정보가 있는 메타데이터 상세화면
  3. format 정보가 있는 컬럼에 1에서 생성한 컬럼 Dictionary를 import
  4. 수정 완료후 조회시 에러가 발생하지 않는것 확인
  5. role이 타임스탬프인 컬럼에서 format을 변경 할 수 있는 아이콘이 출력되지 않는지 확인

Need additional checks?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have added tests to cover my changes.

Additional Context

  • 기존에 format이 있는 컬럼에 컬럼 Dictionary를 import 한 경우 에러가 발생할 때 아래와 같은 조치로 해결할 수 있습니다.
  1. 해당 컬럼의 컬럼 Dictionary를 재설정(또는 해제후 재설정)
  2. 해당 컬럼의 컬럼 Dictionary를 해제후 다른 타입으로 변경
  • 추후 별도의 이슈를 통해 수정이 필요
    메타데이터 > column dictionary를 생성시에 timestamp 타입으로 생성하는 경우 format이 있음
    이렇게 생성한 column dictionary를 메타데이터에서 불러올경우 format validation없이 해당 포맷이 덮어 씌워지게 됨
    (현재는 logical column name을 제외한 값은 변경이 불가능하도록 되어있음)
    추후 format을 변경또는 validation할수 있는것을 활성화 필요

@brandon-wonjune brandon-wonjune added bug Something isn't working @metadata Component Name : Metadata labels Jun 13, 2019
@alchan-lee
Copy link
Contributor

run build
deploy to 1

@alchan-lee alchan-lee added testbed1 Review Test Server 1 to master labels Jun 13, 2019
@AnnieHwang
Copy link
Contributor

@brandon-wonjune

  1. information 아이콘이 원래 포멧을 넣지 않으면 빨간색 경고 벨리데이션이 되었는데 빠졌나요?
    2.시간 포멧을 넣지 않고도 저장되는데 맞나요?
    image

@brandon-wonjune
Copy link
Contributor Author

@AnnieHwang

@brandon-wonjune

  1. information 아이콘이 원래 포멧을 넣지 않으면 빨간색 경고 벨리데이션이 되었는데 빠졌나요?
    2.시간 포멧을 넣지 않고도 저장되는데 맞나요?

혹시 테스트한 데이터가 database, stagingDB 타입 인가요?
데이터소스 타입의 메타데이터가 아니면 format에 대한 validation은 적용되지 않습니다.

@kyungtaak kyungtaak self-requested a review June 13, 2019 03:02
@ufoscw ufoscw merged commit b5a5d09 into 3.2.x Jun 13, 2019
alchan-lee pushed a commit that referenced this pull request Jun 14, 2019
* fn fix format icon css

* fn fix import column dictionary to metadata column error

* fn fix z-index in metadata type list
@alchan-lee alchan-lee deleted the fn-fix_metadata_change_column_error branch June 17, 2019 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working @metadata Component Name : Metadata testbed1 Review Test Server 1 to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants