akka/akka-core

akka-stream log() to accept slf4j Logger

Open

#30,204 opened on Apr 28, 2021

View on GitHub
 (2 comments) (0 reactions) (0 assignees)Scala (13,277 stars) (3,547 forks)batch import
1 - triagedhelp wantedt:stream

Description

Hi Team,

Following discussion in https://github.com/akka/akka-meta/issues/31 and changes in https://github.com/akka/akka/issues/26537 the typed actors now allow passing arround and using slf4j logger for any logging activity. This is nicely documented in: https://doc.akka.io/docs/akka/current/typed/logging.html

This allows much greater flexibility. It is a divergence from the logging mechanism used in classic actors (fine for me) though. But also the current classic actors logging mechanism is used in akka-stream: https://doc.akka.io/japi/akka/2.6/akka/stream/javadsl/Source.html#log(java.lang.String,akka.event.LoggingAdapter) https://doc.akka.io/docs/akka/current/stream/stream-cookbook.html#logging-in-streams

Is there any chance we could get an alternative logging function that would allow passing in your slf4j logger?

Thanks!

Contributor guide

Tech stack
None
Domain
backendobservability
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Akka Streams basicsSLF4J loggingScala programming
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50
Research direction
Examine the current akka stream log() implementation in akka.stream.javadsl.Source and akka.stream.scaladsl.Flow. Study how typed actors now accept a slf4j Logger (issue 26537). Implement a new overload of log() that takes a org.slf4j.Logger instead of akka.event.LoggingAdapter. Ensure consistency with existing logging patterns. Add documentation and tests.