Marsettler

Jaewan Park's blog

한국 카크로치디비 사용자 그룹 GitHub Tags
1 February 2020

2020-02 일지

2020년 2월 일지입니다.

2020-02-13

CockroachDB 이슈 #41274 / sql: Support aggregate functions for statistics

rewrite 플래그를 사용하여 테스트 결과를 자동으로 갱신할 수 있습니다. 결과를 덮어써 버리기 때문에 갱신된 내용을 유심히 살펴보아야 합니다.

`make testlogic TESTFLAGS=-rewrite`

2020-02-09

CockroachDB 이슈 #41274 / sql: Support aggregate functions for statistics

예상보다 예외 상황이 많아 작업시간이 오래 걸렸습니다. 먼저 DISTINCT, FILTER, … 등과 관련된 테스트를 다수 추가했습니다.

DISTINCTORDER BY를 함께 쓰는 이슈는 공통이슈 #43237이기때문에 이번에 수정하지 않을 생각입니다.

TODO: aggTestSpeccolIdxmessage Aggregationcol_idx, filter_col_idx는 복수형으로 변경하는 것이 좋아 보입니다. 이번 작업과는 무관하기에 다른 PR로 수정 예정입니다.

추후 작업에 대한 좋은 템플릿을 만들기 위해 PR을 두개로 나누어달라는 요청이 있어 나눌 생각입니다.


2020-02-05

CockroachDB 이슈 #41274 / sql: Support aggregate functions for statistics

PR #44628에 코멘트가 다수 추가되었습니다. 하나씩 수정해봅니다.

TODO: 작업 중 아래 에러의 의미가 모호해져서 새로운 이슈를 등록할 예정입니다.

return unimplemented.NewWithIssue(28417,
				"aggregate functions with multiple non-constant expressions are not supported",

2020-02-04

CockroachDB 이슈 #41274 / sql: Support aggregate functions for statistics

아래 명령어를 사용하면 aggregate 관련 기능만 빠르게 테스트할 수 있습니다. make test는 너무 오래 걸립니다.

make testlogic FILES=aggregate

2020-02-03

CockroachDB 이슈 #41274 / sql: Support aggregate functions for statistics

아침에 일어나니 생각보다 빠르게 리뷰가 되어 있었습니다. 하지만 수정할 부분이 없다고 생각되어 세부적인 내용을 다시 물어보는 댓글을 남겼습니다. Thank you for the review!는 뭔가 자동완성 같은 느낌이 듭니다.


2020-02-02

CockroachDB 이슈 #41274 / sql: Support aggregate functions for statistics

테스트 작성 중 64비트 float 자료형에서 소수점 끝 두 자리가 잘려서 실패하는 이슈를 확인했습니다. 정확한 원인은 못 찾았지만 일반 컬럼으로 테스트 해본 결과 동일했으며, 테스트 환경에서만 발생했습니다. 의심가는 부분은 go PostgreSQL 클라이언트이지만 추가적인 확인은 하지 않았고, ::decimal 을 사용해 우회했습니다. 나중에 시간이 나면 살펴볼 만한 이슈입니다.


2020-02-01

CockroachDB 이슈 #41274 / sql: Support aggregate functions for statistics

2019년 10월 스프린트 서울에 처음 참가하신 분들께 수정해보시길 권한 이슈입니다. 시간이 조금 흘러도 PR이 제출되지 않아 여러 함수 중 corr()을 직접 작성해보기로 했습니다.

2020년 02월 1일
박재완

tags: diary