apache / amoro

Apache Amoro (incubating) is a Lakehouse management system built on open data lake formats.
https://amoro.apache.org/
Apache License 2.0
747 stars 260 forks source link

[Improvement]: Integrate the log records of debug, info, and error into one file #2955

Open xleoken opened 1 week ago

xleoken commented 1 week ago

Search before asking

What would you like to be improved?

It is not a good practice to record the log records of debug, info, and error separately in different files, and there is a contextual linking relationship between them.

[dcadmin@dcadmin-work logs]$  ll
total 179988
-rw-rw-r-- 1 dcadmin dcadmin         0 Jun 21 16:31 amoro-metrics.log
-rw-rw-r-- 1 dcadmin dcadmin         0 Jun 21 16:31 ams-debug.log
-rw-rw-r-- 1 dcadmin dcadmin  35377462 Jun 21 17:51 ams-error.log
-rw-rw-r-- 1 dcadmin dcadmin 148887752 Jun 21 17:51 ams-info.log
-rw-rw-r-- 1 dcadmin dcadmin         0 Jun 21 16:31 app.log.err
-rw-rw-r-- 1 dcadmin dcadmin     25485 Jun 21 17:51 gc.log.0.current

How should we improve?

No response

Are you willing to submit PR?

Subtasks

No response

Code of Conduct

daragu commented 1 week ago

i can take this :)

zhoujinsong commented 1 week ago

As far as I know, the ams-info.log file now also includes warning and error logs. Similarly, the ams-debug.log file contains all logs(disable by default).

xleoken commented 1 week ago

@daragu thanks for taking this, but we are still discussing this issue.