Skip to content

Update CommonUtil.java#3932

Merged
lmsurpre merged 3 commits into
LinuxForHealth:mainfrom
berkant-k:patch-2
Aug 31, 2022
Merged

Update CommonUtil.java#3932
lmsurpre merged 3 commits into
LinuxForHealth:mainfrom
berkant-k:patch-2

Conversation

@berkant-k
Copy link
Copy Markdown
Contributor

When logLevel set to FINE; castings to URLClassLoader was throwing java.lang.ClassCastException.
To fix this issue logClassPath method changed

Signed-off-by: Berkant Karduman karduman.berkant@gmail.com

When logLevel set to FINE;  castings to URLClassLoader  was throwing  java.lang.ClassCastException.
To fix this issue logClassPath method changed

Signed-off-by: Berkant Karduman <karduman.berkant@gmail.com>
classpath directly logged

Signed-off-by: Berkant Karduman <karduman.berkant@gmail.com>
variable name changed

Signed-off-by: Berkant Karduman <karduman.berkant@gmail.com>
Copy link
Copy Markdown
Member

@lmsurpre lmsurpre left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @berkant-k!

@lmsurpre lmsurpre merged commit 19f1a83 into LinuxForHealth:main Aug 31, 2022
berkant-k added a commit to berkant-k/FHIR that referenced this pull request Sep 3, 2022
* Update CommonUtil.java

When logLevel set to FINE;  castings to URLClassLoader  was throwing  java.lang.ClassCastException.
To fix this issue logClassPath method changed

Signed-off-by: Berkant Karduman <karduman.berkant@gmail.com>

* Update CommonUtil.java

classpath directly logged

Signed-off-by: Berkant Karduman <karduman.berkant@gmail.com>

* Update CommonUtil.java

variable name changed

Signed-off-by: Berkant Karduman <karduman.berkant@gmail.com>

Signed-off-by: Berkant Karduman <karduman.berkant@gmail.com>
Signed-off-by: Berkant KARDUMAN <berkant.karduman@tigahealth.com>
@berkant-k berkant-k deleted the patch-2 branch September 3, 2022 04:13
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