-
Notifications
You must be signed in to change notification settings - Fork 328
HAWQ-1276. The error message is not friendly when ranger plugin servi… #1088
Conversation
…ce is unavailable.
@@ -366,7 +365,7 @@ static size_t write_callback(char *contents, size_t size, size_t nitems, | |||
if (curl->response.buffer == NULL) | |||
{ | |||
/* out of memory! */ | |||
elog(WARNING, "not enough memory for Ranger response"); | |||
elog(WARNING, "cannot allocate memory for ranger response"); |
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.
Why do you modify the out of memory error message. I think the original one is more specified, and may be better.
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.
hmm, IMO , not enough memory is not the only error that will occur. There maybe other errors in the repalloc implementation. So I changed the messages.
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.
So should the comment be consistent with the code?
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.
sure. fixed.
+1 |
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.
LGTM.
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.
LGTM. +1
@@ -294,7 +294,7 @@ static json_object *create_ranger_request_json(List *request_list, List *result_ | |||
break; | |||
} | |||
default: | |||
elog(ERROR, "unrecognized objkind: %d", (int) kind); | |||
elog(ERROR, "unrecognized object kind : %d", (int) kind); |
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.
since this is a message to users, we should map this "kind" to string, so that users can know which kind is not supported.
merged into master. closed. |
…ce is unavailable.