Update PrestoS3FileSystem.java#403
Conversation
The `LOG.debug("destination does not exist");` should be `log.debug("destination does not exist");`, because the `LOG` is parent FileSystem's reference.
|
Thanks for your pull request. The authors of the following commits have not signed the Contributor License Agreement (CLA): 83e1395 | Update PrestoS3FileSystem.java The Please check the FAQs first. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ShefronYudy The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome @ShefronYudy! |
|
/sync |
|
@Nitin-Kashyap: sync pull request is restricted to collaborators DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the opensourceways/test-infra repository. |
The
LOG.debug("destination does not exist");should belog.debug("destination does not exist");, because theLOGis parent FileSystem's reference.What type of PR is this?
bug
What does this PR do / why do we need it:
When use LOG to print some messages, The log class of log message is not the current
PrestoS3FileSystem, but the parent FileSystem.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewers: