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

BUG-50968 PROJ-2765 Checkpoint Scale 매뉴얼 수정 #557

Open
wants to merge 69 commits into
base: master
Choose a base branch
from

Conversation

mason-lee-alti
Copy link
Collaborator

하위 호환 불가이기 때문에 trunk 에만 적용되는 프로젝트 입니다.

추가 되어야 하는 이미지는 아래 페이지에 첨부했습니다.
http://nok.altibase.com/pages/viewpage.action?pageId=73209837

@Soyoon-Choi Soyoon-Choi changed the title Manual 수정 초안 PROJ-2765 Checkpoint Scale 매뉴얼 수정 Jun 17, 2024
@Soyoon-Choi Soyoon-Choi removed the request for review from applys June 20, 2024 23:47
@Soyoon-Choi Soyoon-Choi requested a review from eheejung June 28, 2024 04:40
-p 옵션 내용 수정
Copy link
Collaborator

@eheejung eheejung left a comment

Choose a reason for hiding this comment

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

리뷰 남깁니다.
매체복구 시 주의사항 부분을 처음에 제대로 안 봐서;;; 리뷰 내용이 좀 있습니다.

Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
Manuals/Altibase_trunk/kor/Administrator's Manual.md Outdated Show resolved Hide resolved
@Soyoon-Choi Soyoon-Choi requested a review from eheejung October 8, 2024 01:04
Copy link
Collaborator

@eheejung eheejung left a comment

Choose a reason for hiding this comment

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

리뷰 남깁니다.
CHECKPOINT_SCALE_SINGLE_DW_BUFFER_SIZE 부분에서 정현님에게 질문 남긴 부분만 확인하고 approve 하겠습니다.

@Soyoon-Choi Soyoon-Choi requested a review from eheejung October 17, 2024 01:08
@Soyoon-Choi
Copy link
Collaborator

Soyoon-Choi commented Oct 17, 2024

모든 수정 마쳤습니다. 최종적으로 검토 부탁드립니다. 이후 영문 매뉴얼 작업은 따로 버그를 열어 진행하겠습니다.

@Soyoon-Choi Soyoon-Choi requested a review from lswhh October 17, 2024 01:09
@Soyoon-Choi Soyoon-Choi requested a review from eheejung October 21, 2024 01:48
dumpla 내용 수정
CHECKPOINT_SCALE_SINGLE_DW_BUFFER_SIZE 내용 수정
@Soyoon-Choi
Copy link
Collaborator

Soyoon-Choi commented Dec 3, 2024

@mason-lee-alti
CHECKPOINT_SCALE_SINGLE_DW_BUFFER_SIZE 의 내용은 @eheejung 님과 검토 결과 간단하게 작성하기로 결정했습니다.

"더티 페이지 스냅샷의 크기가 체크포인트 이미지 파일 크기의 50% 이하일지라도, 설정된 값보다 크면 풀 라이트 방식으로 체크포인트를 수행하게 된다." 이 내용을 삭제하게 된 근거는 다음과 같습니다.

시나리오)
CHECKPOINT_SCALE_SINGLE_DW_BUFFER_SIZE의 설정값이 A일 때 각각 다음 경우의 체크포인트 이미지 파일이 있는 경우:

  • CASE 1 A*2 < 체크포인트 이미지 파일 크기: 위 조건에 걸려 더블 라이트 동작이 불발할 가능성 있음
  • CASE 2 A*2 = 체크포인트 이미지 파일 크기: 더블 라이트 동작 시 이 프로퍼티의 영향을 받지 않음
  • CASE 3 A*2 > 체크포인트 이미지 파일 크기: 은 더블 라이트 동작 시 이 프로퍼티의 영향을 받지 않음
    즉 이 프로퍼티는 CASE 1, CHECKPOINT_SCALE_SINGLE_DW_BUFFER_SIZE에 설정된 크기의 2배보다 큰 체크포인트 이미지 파일이 있을 때에만 비효율적으로 동작하게 됩니다.
    이 때 매뉴얼에서 설명하는 대로 사용자가 가장 큰 체크포인트 이미지 파일의 반으로 이 값을 설정한다면 CASE 1의 상황이 애초에 발생하지 않게 됩니다.
    더불어 풀 라이트 방식으로 동작하는 조건 중 "스냅샷의 크기가 이 프로퍼티의 설정보다 작을 때"라는 설명을 매뉴얼에 마땅히 삽입할 곳이 없는 점을 고려하여,
    "더티 페이지 스냅샷의 크기가 체크포인트 이미지 파일 크기의 50% 이하일지라도, 설정된 값보다 큰" 상황에 대한 설명을 전면 삭제하기로 했습니다.
    대신, 가장 큰 체크포인트 이미지 파일의 절반 크기로 설정할 것을 권고하는 것이 아닌 필수 설정 사항으로 설명하여 매뉴얼을 따르면 효율에 손해를 보지 않도록 안내했습니다.
  • 이후 만약 해당 기능이 릴리즈되어 이 프로퍼티 설정으로 인한 성능 관련 인시던트가 잦을 경우, General Reference의 프로퍼티 설정에 효율적으로 이 프로퍼티를 설정하는 법, 혹은 이유에 관해 짧게 추가하도록 정리했습니다.

더불어 최종 검토 중, Utilities Manual의 dumpla에도 수정이 필요하여 해당 부분을 작성하였습니다.
마지막으로 검토 부탁드립니다.
이후 이 PR의 변경 사항을 영문 매뉴얼에 적용하는 작업은 PR#614에서 진행합니다.

Copy link
Collaborator

@eheejung eheejung left a comment

Choose a reason for hiding this comment

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

장기간 수고 많으셨습니다!

@Soyoon-Choi
Copy link
Collaborator

Soyoon-Choi commented Dec 10, 2024

12/10 정현 님에게 이메일로 추가 수정이 필요 없다는 답변을 받았습니다.

      1. 오후 3:13
        안녕하세요, 이정현 님

QA실 TW 최소윤입니다.

(중략)

최종 리뷰를 부탁드리며, 추가로 수정이 필요한 사항이 있다면 언제든 말씀 주시면 신속히 반영하겠습니다.

항상 감사드립니다.

최소윤 드림

오전 10:16
안녕하세요. 이정현입니다.

리뷰 확인했습니다. 추가 수정이 필요해보이지 않습니다.

감사합니다.

이에 따라 @scott-altibase님의 승인이 나는 대로 Master 브랜치에 머지하고, 영문 매뉴얼의 리뷰를 진행하겠습니다.

Copy link
Contributor

@junhyungjin junhyungjin left a comment

Choose a reason for hiding this comment

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

코멘트 남겼습니다

@@ -7595,7 +7630,13 @@ Altibase는 핑퐁 체크포인트 방식을 사용하기 때문에 각 데이

##### CURRENT_DB

체크포인트 시 더티 페이지 (Dirty Page, 변경된 페이지)가 내려가는 데이터베이스 이미지 파일 그룹으로 0 혹은 1 값을 가진다.
테이블스페이스의 안정적인 체크포인트 이미지 파일의 핑퐁 번호를 나타낸다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
테이블스페이스의 안정적인 체크포인트 이미지 파일의 핑퐁 번호를 나타낸다.
테이블스페이스에 존재하는 안정적인 체크포인트 이미지 파일의 핑퐁 번호를 나타낸다.


BEGIN SNAPSHOT을 할 때의 시점을 기준 SNAPSHOT SCN으로 설정하고, 설정된 SCN을
기준으로 iLoader를 사용하여 데이터를 EXPORT한다.

*checkpoint_scale_clause*

데이터베이스의 체크포인트 스케일을 변경한다. 이 절은 Altibase 서버 구동 순서 중 CONTROL 단계에서만 수행할 수 있다. 체크포인트 스케일을 페어에서 싱글로 변경하면 체크포인트 이미지 한 쌍 중 불안정한 체크포인트 이미지 파일을 삭제하며, 싱글에서 페어로 변경하면 체크포인트 이미지 파일을 복사하여 한 쌍으로 만든다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
데이터베이스의 체크포인트 스케일을 변경한다. 이 절은 Altibase 서버 구동 순서 중 CONTROL 단계에서만 수행할 수 있다. 체크포인트 스케일을 페어에서 싱글로 변경하면 체크포인트 이미지 한 쌍 중 불안정한 체크포인트 이미지 파일을 삭제하며, 싱글에서 페어로 변경하면 체크포인트 이미지 파일을 복사하여 한 쌍으로 만든다.
데이터베이스의 체크포인트 스케일을 변경한다. 이 구문은 Altibase 서버 구동 순서 중 CONTROL 단계에서만 수행할 수 있다. 체크포인트 스케일을 페어에서 싱글로 변경하면 체크포인트 이미지 한 쌍 중 불안정한 체크포인트 이미지 파일을 삭제하며, 싱글에서 페어로 변경하면 체크포인트 이미지 파일을 복사하여 한 쌍으로 만든다.

Copy link
Collaborator

Choose a reason for hiding this comment

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

확인해보니 "이 절은"과 "이 구문은"이 혼용되고 있습니다. 이후 버그를 열어 통일시키는 것이 나아보입니다.

Manuals/Altibase_trunk/kor/Utilities Manual.md Outdated Show resolved Hide resolved
Soyoon-Choi and others added 3 commits December 11, 2024 09:27
Co-authored-by: Hyung jin Jun <junhyungjin@gmail.com>
…nary.md

Co-authored-by: Hyung jin Jun <junhyungjin@gmail.com>
Change Header 1 syntax
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.

4 participants