apache / apache/pulsar-client-cpp

Custom LoggerFactory for log4cxx

オープン
#376 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
C++
スター
71
フォーク
90
平均マージ
2時間 33分
マージ済み PR(30日)
3

説明

FileLoggerFactory simply outputs logs to a single file.
If users want to rotate log files, they need to implement custom LoggerFactory by themselves.
Log4cxxLogger was deleted in #283, but this is useful for users.

So I would like to recover Log4cxxLogger.
I would not create specialized methods for Log4cxxLogger as there used to be, but would like to make it one of the options in the same category as ConsoleLoggerFactory and FileLoggerFactory.
https://github.com/apache/pulsar-client-cpp/pull/283/files#diff-abb02af9369945477ebc3329b299544ba67e214ac3f36c55b1f9e27f68469abf
If we want to avoid embedding it as a build option, we can separate the client and logger packages.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず #283 の diff と現在の ConsoleLoggerFactory および FileLoggerFactory の実装を確認し、logger のオプションがどのように公開されているかを確認します。組み込みの build オプションにするか、client/logger を分離した package にするかを選択する前に、build と package の構成を確認します。Log4cxxLogger が同等のオプションとして復元され、ユーザーが専用メソッドなしでカスタム log rotation に利用できれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp
領域
build-system, observability-sre
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
30/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。