[CBRD-25541] Fixed unnecessary heap operations in show access status by passing NULL to the class_oid parameter

[InChiJun]

http://jira.cubrid.org/browse/CBRD-25541

• Purpose
  • show access status 수행 시 css_make_access_status_exist_user() 함수에서 heap_scancache_start(), heap_next() 함수를 호출함.
  • 두 함수 호출 시 class_oid 파라미터에 NULL을 전달하고 있음.
    • 이 경우 class_oid를 찾는 연산을 다시 수행해야 함.
  • css_make_access_status_exist_user() 함수에서 이미 class_oid를 인자로 전달받았기 때문에 class_oid를 두 함수에 전달함으로써 불필요하게 다시 class_oid를 찾는 연산을 제거해야 함
• Implementation
  • heap_scancache_start(), heap_next() 함수에서 class_oid 파라미터에 class_oid 값을 전달하게 함.
• Remarks
  • class_oid가 NULL이 아닌 정상적인 값으로 전달되어야 아래에서 수행된 최적화가 적용될 수 있음.
    • [CBRD-25078] [Regression] Access status result was wrong print #5394 (comment)
    • http://jira.cubrid.org/browse/CBRD-25037?focusedCommentId=4767502&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-4767502

[hornetmj]

heap_scancache_start() 함수를 통해 scan_cache->node.class_oid에 class_oid 정보가 저장됩니다. heap_next_internal 함수를 보면 scan_cache->node.class_oid 정보를 체크하도록 되어 있습니다. 그럼에도 불구하고 heap_next()에 NULL이 아닌 class_oid 정보를 넘겨주어야 하는 이유가 있을까요?

[InChiJun]

NULL값을 넘겨주게 되면 8151 라인에서 *next_oid에 oid를 할당해주는 작업을 하고, heap연산을 다시 수행합니다.
하지만, class_oid의 값을 넘겨주게 되면 추가로 할당해주는 작업 없이 heap 연산을 수행하므로 불필요한 연산을 하지 않게 됩니다.

[InChiJun]

정정합니다. heap_scancache_start() 함수에서 class_oid 값이 저장되기 때문에 heap_next() 함수에 class_oid를 NULL로 전달하여도 class_oid를 식별하기에는 문제가 없습니다.
2777 라인은 변경하지 않도록 코드 반영하겠습니다.