Skip to content

Conversation

Sanket-Shelar
Copy link
Contributor

What changes were proposed in this pull request?

Shared Resource API /dataset/{id}/resources/{serviceName} is failing for Oracle database as it is using distinct with CLOB type in query.
Additionally whenever GDS resource APIs are invoked, there are logs in error as SubResourceMasks and DefaultTagMasks are null.

How was this patch tested?

Build Successful. API tested in oracle and checked logs and error logs were not present.

return getMapper().readValue(jsonStr, TYPE_LIST_RANGER_GDS_MASK_INFO);
} catch (Exception e) {
LOG.error("Cannot get Map<String, RangerPolicyItemDataMaskInfo> from {}", jsonStr, e);
LOG.error("Cannot get List<RangerGdsMaskInfo> from {}", jsonStr, e);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should there be null check on this "if (StringUtils.isNotEmpty(jsonStr)) " and then do readValue to avoid excetion?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update the Patch. kindly review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants