akka/akka-http

Split packages in akka.http.core and akka.http

Open

#2,903 opened on Jan 21, 2020

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Scala (1,311 stars) (598 forks)batch import
1 - triagedbinary-incompatiblehelp wantednice-to-have (low-prio)

Description

Akka HTTP Version 10.1.11:

The following packages currently occur in akka.http.core and akka.http with disjunct classes:

  • akka.http.impl.settings
  • akka.http.javadsl.settings
  • akka.http.scaladsl.settings

We have various options here:

  • Move all classes to the same package into the same artifact
  • Rename one of the packages to avoid the split-package issue, preferably choosing a package name which shares the artifact name as it's prefix
  • With a very good reason (is that even possible?) I think we can keep the split package, but it is the most painful and dirty solution

Related to https://github.com/woq-blended/akka-osgi/issues/2. Comments of https://github.com/akka/akka-http/issues/2902 apply.

Contributor guide

Tech stack
scala
Domain
backend
Issue type
refactor
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.
mostly clear
Prerequisites
ScalaAkka HTTPOSGisbt
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
25
Research direction
The issue proposes three options to resolve split packages between akka.http.core and akka.http artifacts. The preferred approach is to move classes to the same package into the same artifact or rename one package to avoid split packages. Examine the existing package layout in the source code (e.g., akka http core/src/main/scala/akka/http/impl/settings) and determine which option is least disruptive. Also review comments in referenced issue #2902 and the OSGi issue linked for further context.
Split packages in akka.http.core and akka.http · akka/akka-http#2903 | Good First Issue