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

Spotlake : 데이터 수정 검토 필요 #481

Open
Kim-Yul opened this issue Oct 30, 2024 · 4 comments
Open

Spotlake : 데이터 수정 검토 필요 #481

Kim-Yul opened this issue Oct 30, 2024 · 4 comments
Assignees
Labels

Comments

@Kim-Yul
Copy link

Kim-Yul commented Oct 30, 2024

Sandevistan 기능 추가를 위해 데이터를 보다 보니, 랜딩화면에서의 AZ 데이터 형식과 TSDB에 저장되어 있는 AZ 데이터 형식이 달라 결과적으로 다른 출력값을 보이는 걸 확인하였습니다.
이를 수정할지, 그대로 둘지 검토가 필요합니다.

랜딩화면에서의 데이터 저장 형식
image

TSDB에 저장되어 있는 데이터 저장 형식
image

@Kim-Yul
Copy link
Author

Kim-Yul commented Oct 31, 2024

확인해보니 TSDB 에는 AZ의 풀네임이 제대로 저장되어 있었습니다. 코드를 비교하여 확인해보았을 때, API에서 의도적으로 AZ의 네임을 숫자로 변경하여 출력하고 있는 것을 발견하였습니다. 이전에 Spotlake 개발 시 의도가 있었던 것으로 유추됩니다.

TSDB 내 쿼리 결과
image

AZ 형식을 숫자로 변경하는 API의 코드

    # If the column is AZ
    elif 'Name' in info and info['Name'] == 'AZ':
        return _parse_column_name(info) + datum['ScalarValue'].split("-az")[-1] + '"'

현재 진행할 수 있는 방향은 3가지 입니다.

  • raw_data의 AZ 형식을 숫자로 바꾸기 (기존 : apse2-az3 -> 변경 : 3)
  • API 의 AZ 형식을 풀네임으로 바꾸기 (기존 : 3 -> 변경 : apse2-az3)
  • 현재 상태를 유지하기

어떤 것이 더 좋을지는 검토 후에 결정하도록 하겠습니다.

@kmu-leeky
Copy link
Member

유림아. 이건 일부러 바꾼걸로 기억해. az 값이 region 에도 있으면 중복되니 그래서 az 값은 숫자로만 남겼던것 같아.

@Kim-Yul
Copy link
Author

Kim-Yul commented Nov 1, 2024

확인했습니다! 그러면 현재 출력 형식을 그대로 유지하는 것으로 하겠습니다.

@Kim-Yul Kim-Yul closed this as completed Nov 1, 2024
@Kim-Yul Kim-Yul reopened this Nov 4, 2024
@Kim-Yul
Copy link
Author

Kim-Yul commented Nov 4, 2024

기존의 2개의 기능을 합쳐 새로 만들어질 collector 에서 해당 내용 중 raw_data의 AZ 형식을 숫자로 바꾸는 작업을 진행합니다.
작업이 완료되면 클로즈하겠습니다.

raw_data의 AZ 형식을 숫자로 바꾸기 (기존 : apse2-az3 -> 변경 : 3)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants