-
Notifications
You must be signed in to change notification settings - Fork 124
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
[CBRD-25707] With the addition of [user_schema] to SP, the --no-user-specified-name option of the loaddb utility has been fixed to function correctly, even when SPs are included in unload files from the past. #5659
base: feature/plcsql-p1n
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20250,6 +20250,98 @@ do_find_synonym_by_query (const char *name, char *buf, int buf_size) | |
#undef QUERY_BUF_SIZE | ||
} | ||
|
||
int | ||
do_find_stored_procedure_by_query (const char *name, char *buf, int buf_size) | ||
{ | ||
#define QUERY_BUF_SIZE 2048 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 다른 함수들과 동일하게 함수 내에서 매크로를 정의하고 제거하시는데, 상수를 사용하는 것이 아니라 매크로를 따로 사용하시는 이유가 무엇인지 알 수 있을까요? |
||
DB_QUERY_RESULT *query_result = NULL; | ||
DB_QUERY_ERROR query_error; | ||
DB_VALUE value; | ||
const char *query = NULL; | ||
char query_buf[QUERY_BUF_SIZE] = { '\0' }; | ||
const char *current_schema_name = NULL; | ||
const char *sp_name = NULL; | ||
int error = NO_ERROR; | ||
|
||
db_make_null (&value); | ||
query_error.err_lineno = 0; | ||
query_error.err_posno = 0; | ||
|
||
if (name == NULL || name[0] == '\0') | ||
{ | ||
ERROR_SET_WARNING (error, ER_OBJ_INVALID_ARGUMENTS); | ||
return error; | ||
} | ||
|
||
assert (buf != NULL); | ||
|
||
current_schema_name = sc_current_schema_name (); | ||
|
||
sp_name = sm_remove_qualifier_name (name); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. sp_name이 NULL일 경우는 체크할 필요가 없을까요? |
||
query = "SELECT [unique_name] FROM [%s] WHERE [sp_name] = '%s' AND [owner].[name] != UPPER ('%s')"; | ||
assert (QUERY_BUF_SIZE > snprintf (NULL, 0, query, CT_STORED_PROC_NAME, sp_name, current_schema_name)); | ||
snprintf (query_buf, QUERY_BUF_SIZE, query, CT_STORED_PROC_NAME, sp_name, current_schema_name); | ||
assert (query_buf[0] != '\0'); | ||
|
||
error = db_compile_and_execute_local (query_buf, &query_result, &query_error); | ||
if (error < NO_ERROR) | ||
{ | ||
ASSERT_ERROR (); | ||
goto end; | ||
} | ||
|
||
error = db_query_first_tuple (query_result); | ||
if (error != DB_CURSOR_SUCCESS) | ||
{ | ||
if (error == DB_CURSOR_END) | ||
{ | ||
ERROR_SET_WARNING_1ARG (error, ER_SP_NOT_EXIST, name); | ||
} | ||
else | ||
{ | ||
ASSERT_ERROR (); | ||
} | ||
|
||
goto end; | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 다른 에러가 발생할 경우는 없나요? |
||
|
||
error = db_query_get_tuple_value (query_result, 0, &value); | ||
if (error != NO_ERROR) | ||
{ | ||
ASSERT_ERROR (); | ||
goto end; | ||
} | ||
|
||
if (!DB_IS_NULL (&value)) | ||
{ | ||
assert (strlen (db_get_string (&value)) < buf_size); | ||
strcpy (buf, db_get_string (&value)); | ||
} | ||
else | ||
{ | ||
/* unique_name must not be null. */ | ||
ASSERT_ERROR_AND_SET (error); | ||
goto end; | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. if (DB_IS_NULL) 로 하는 게 읽기 편할 것 같습니다. |
||
|
||
error = db_query_next_tuple (query_result); | ||
if (error != DB_CURSOR_END) | ||
{ | ||
/* No result can be returned because unique_name is not unique. */ | ||
buf[0] = '\0'; | ||
} | ||
|
||
end: | ||
if (query_result) | ||
{ | ||
db_query_end (query_result); | ||
query_result = NULL; | ||
} | ||
|
||
return error; | ||
#undef QUERY_BUF_SIZE | ||
} | ||
|
||
/* | ||
* do_set_trace_to_query_flag() - | ||
* return: void | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -67,6 +67,7 @@ | |
#include "oid.h" | ||
#include "string_buffer.hpp" | ||
#include "db_value_printer.hpp" | ||
#include "execute_statement.h" | ||
|
||
#define PT_NODE_SP_NAME(node) \ | ||
(((node)->info.sp.name == NULL) ? "" : \ | ||
|
@@ -154,6 +155,7 @@ jsp_find_stored_procedure (const char *name, DB_AUTH purpose) | |
DB_VALUE value; | ||
int save, err = NO_ERROR; | ||
char *checked_name; | ||
char other_class_name[DB_MAX_IDENTIFIER_LENGTH]; | ||
|
||
if (!name) | ||
{ | ||
|
@@ -169,8 +171,29 @@ jsp_find_stored_procedure (const char *name, DB_AUTH purpose) | |
if (er_errid () == ER_OBJ_OBJECT_NOT_FOUND) | ||
{ | ||
er_clear (); | ||
err = ER_SP_NOT_EXIST; | ||
er_set (ER_WARNING_SEVERITY, ARG_FILE_LINE, err, 1, checked_name); | ||
|
||
/* This is the case when the loaddb utility is executed with the --no-user-specified-name option as the dba user. */ | ||
if (db_get_client_type () == DB_CLIENT_TYPE_ADMIN_LOADDB_COMPAT) | ||
{ | ||
other_class_name[0] = '\0'; | ||
|
||
err = do_find_stored_procedure_by_query (name, other_class_name, DB_MAX_IDENTIFIER_LENGTH); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. err 처리는 안해도 되나요? |
||
if (other_class_name[0] != '\0') | ||
{ | ||
db_make_string (&value, other_class_name); | ||
mop = db_find_unique (db_find_class (SP_CLASS_NAME), SP_ATTR_UNIQUE_NAME, &value); | ||
if (er_errid () == ER_OBJ_OBJECT_NOT_FOUND) | ||
{ | ||
err = ER_SP_NOT_EXIST; | ||
er_set (ER_WARNING_SEVERITY, ARG_FILE_LINE, err, 1, other_class_name); | ||
} | ||
} | ||
} | ||
else | ||
{ | ||
err = ER_SP_NOT_EXIST; | ||
er_set (ER_WARNING_SEVERITY, ARG_FILE_LINE, err, 1, checked_name); | ||
} | ||
Comment on lines
+178
to
+196
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 클라이언트 타입이 일반적인 경우가 아닌 LOADDB에서만 동작하므로 별도의 함수로 분리하고, other_class_name이 해당 함수 호출 시에 생성되도록 하면 좋을 것 같습니다. 가독성 면에서도 주요 로직을 파악하기에 함수로 분리하는 것이 좋아보입니다. |
||
} | ||
|
||
if (mop) | ||
|
@@ -1435,8 +1458,6 @@ jsp_check_stored_procedure_name (const char *str) | |
char tmp[SM_MAX_IDENTIFIER_LENGTH + 2]; | ||
char *name = NULL; | ||
static int dbms_output_len = strlen ("dbms_output."); | ||
char owner_name[DB_MAX_USER_LENGTH]; | ||
owner_name[0] = '\0'; | ||
|
||
|
||
if (strncasecmp (str, "dbms_output.", dbms_output_len) == 0) | ||
|
@@ -1449,8 +1470,6 @@ jsp_check_stored_procedure_name (const char *str) | |
sm_user_specified_name (str, buffer, SM_MAX_IDENTIFIER_LENGTH); | ||
} | ||
|
||
// sm_downcase_name (str, buffer, SM_MAX_IDENTIFIER_LENGTH); | ||
|
||
name = strdup (buffer); | ||
|
||
return name; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
쿼리 내용을 보니 이름은 갖고 소유자는 현재 스키마와 다른 경우를 찾는 것 같습니다.
함수 이름에 이런 사정이 잘 드러나지 않는 듯 합니다.
함수 이름으로 do_find_sp_of_another_owner 정도가 어떨까요.