[Feat] 마이페이지 인사이트 조회 #159
[Feat] 마이페이지 인사이트 조회 #159
Conversation
scripts/batch_deploy.sh
Outdated
| JAR_DIR='/home/ubuntu/'$2 | ||
|
|
||
| cp keewe-batch/build/libs/keewe-batch-0.0.1-SNAPSHOT.jar ${JAR_DIR}/ | ||
|
|
||
| echo "> Auto batch deploy starting...." | ||
|
|
||
| mkdir $JAR_DIR | ||
| mkdir -p $JAR_DIR | ||
| cd $JAR_DIR | ||
|
|
||
| ORIGIN_JAR_PATH= "${JAR_DIR}/*.jar" | ||
| chmod 770 ${JAR_DIR}/$1-0.0.1-SNAPSHOT.jar | ||
| ORIGIN_JAR_PATH="${JAR_DIR}/$1-0.0.1-SNAPSHOT.jar" | ||
| ORIGIN_JAR_NAME=$(basename ${ORIGIN_JAR_PATH}) | ||
| TARGET_PATH="${JAR_DIR}/batch-application.jar" | ||
|
|
||
| echo "> Do deploy :: $1" | ||
| echo " > 배포 JAR: "${ORIGIN_JAR_NAME} | ||
|
|
||
| echo " > sudo ln -s -f ${JAR_DIR}${ORIGIN_JAR_NAME} ${TARGET_PATH}" | ||
| sudo ln -s -f ${JAR_DIR}${ORIGIN_JAR_NAME} ${TARGET_PATH} | ||
| echo " > sudo ln -s -f ${JAR_DIR}/${ORIGIN_JAR_NAME} ${TARGET_PATH}" | ||
| ln -s -f "${JAR_DIR}/${ORIGIN_JAR_NAME} ${TARGET_PATH}" | ||
|
|
||
| echo "> Batch deploy Complete ! !" | ||
| echo "> Batch deploy Complete ! !" |
There was a problem hiding this comment.
되기는 하는데 다른 데서도 안붙여서 떼는게 좋아 보여요
There was a problem hiding this comment.
보니까 젠킨스에서 파라미터 주입하게돼있던데 파라미터 혹시 잘못넣으면 걱정돼서요 ㅎ
There was a problem hiding this comment.
아하 거기까진 생각 못했네요
|
|
||
| public List<Insight> findByUserIdAndDrawerId(Long userId, Long drawerId, CursorPageable<Long> cPage) { | ||
| return queryFactory | ||
| .select(insight) | ||
| .from(insight) | ||
| .where(insight.writer.id.eq(userId) | ||
| .and(insight.id.lt(cPage.getCursor())) | ||
| .and(drawerIdEq(drawerId)) | ||
| ) | ||
| .orderBy(insight.id.desc()) | ||
| .limit(cPage.getLimit()) | ||
| .fetch(); | ||
| } | ||
|
|
||
|
|
||
| private BooleanExpression drawerIdEq(Long drawerId) { | ||
| return drawerId != null ? insight.drawer.id.eq(drawerId) : null; | ||
| } |
There was a problem hiding this comment.
drawerId가 Null인 케이스는 서랍 탭 중에 전체인 경우죠?
그럼 조건이 equal null로 들어가면 전체 조회가 안되지 않을까여?
There was a problem hiding this comment.
네 drawerId가 null이면 전체인 경우에요
저렇게 사용하면 drawerId가 null인 경우 and(null)로 들어가면 파라미터가 null로 들어가서 조건문에서 제외돼요.
그래서 drawerId가 null인 경우에 아래처럼 쿼리가 나갑니다.
where
insight0_.writer_id=?
and insight0_.insight_id<?
There was a problem hiding this comment.
아.. 그냥 뭉탱이로 Null이면 상관없나요
There was a problem hiding this comment.
네 querydsl의 where에서 파라미터가 null인 경우에는 제외한다고 하네요
| @Transactional(readOnly = true) | ||
| public List<InsightMyPageDto> getByUserId(User user, Long targetUserId, Long drawerId, CursorPageable<Long> cPage) { | ||
| List<Insight> insights = insightQueryRepository.findByUserIdAndDrawerId(targetUserId, drawerId, cPage); |
There was a problem hiding this comment.
List를 가져오는것과 단일객체를 가져올때 네이밍 구분이 명확했음 좋겠어요
이름은 userId로만 가져오는 것처럼 단순해보이지만 실제 파라미터는 너무 많아서 그렇지 않은거같기도하구요.ㅎ
There was a problem hiding this comment.
서랍이 추가되기도 하면서 이름이 알맞지 않아보이네요
getInsightsForMyPage로 바꾸겠습니다
|
수고하셨습니당 |
마이 페이지 인사이트 조회
batch_deploy 스크립트 수정