doris icon indicating copy to clipboard operation
doris copied to clipboard

[Enhancement] Use `NIO` to replace the legacy IO.

Open jackwener opened this issue 3 years ago • 5 comments

Search before asking

  • [X] I had searched in the issues and found no similar issues.

Description

The legacy IO is hard to use. After Java 7, the NIO is imported, it's easy to use and elegant.

Old:
File file = new File("...");
file.createNewFile()
mkdir()

New:
Path path = Paths.get("");
......

like this https://github.com/apache/doris/pull/11645/files#diff-470c2ed9f28730c3d42f0a11fed56f0a4dae030a9eb9c7521c9793ed6499ec8d

Solution

No response

Are you willing to submit PR?

  • [ ] Yes I am willing to submit a PR!

Code of Conduct

jackwener avatar Aug 14 '22 15:08 jackwener

example: #11645

jackwener avatar Aug 18 '22 13:08 jackwener

@jackwener Hi jack, can i help this issue?

dinggege1024 avatar Aug 30 '22 02:08 dinggege1024

Yes, if have any question, feel free @me

DingGeGe @.***> 于 2022年8月30日周二 10:55写道:

@jackwener https://github.com/jackwener Hi jack, can i help this issue?

— Reply to this email directly, view it on GitHub https://github.com/apache/doris/issues/11768#issuecomment-1231081435, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHI4SLMORPPZQOBDLLC43G3V3VZY5ANCNFSM56QAQXHA . You are receiving this because you were mentioned.Message ID: @.***>

jackwener avatar Aug 30 '22 02:08 jackwener

Yes, if have any question, feel free @me DingGeGe @.> 于 2022年8月30日周二 10:55写道: @jackwener https://github.com/jackwener Hi jack, can i help this issue? — Reply to this email directly, view it on GitHub <#11768 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHI4SLMORPPZQOBDLLC43G3V3VZY5ANCNFSM56QAQXHA . You are receiving this because you were mentioned.Message ID: @.>

What's the Enhancement Scope, EP: Which module OR path for legacy IO I should enhance OR anywhere in this Doris project~

dinggege1024 avatar Aug 31 '22 05:08 dinggege1024

https://github.com/apache/doris/pull/12818

@jackwener Hi,plz help review this pr plz

dinggege1024 avatar Sep 20 '22 08:09 dinggege1024