evolution-gaming / kafka-journal / 8249405725
94%

DEFAULT BRANCH: master
Ran
Jobs 1
Files 191
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 81.762% (-0.1%) from 81.91%
8249405725

push

web-flow 
Move `CassandraConsistencyConfig` from `eventual.cassandra` to `cassandra` package(#594)

**Reasoning**

As part of restoring binary compatibility with a previous version, I am
going to copy-paste several classes such as `CassandraHealthCheck`,
`SettingsCassandra`, `SettingsStatements` and related classes to
`eventual-cassandra` project, so they could depend on a restored
`EventualCassandraConfig.ConsistencyConfig`.

To make sure there is no conflict between the two copies, I will move
the classes in `cassandra` project from `eventual.cassandra` to
`cassandra` package instead.

The classes from `eventual-cassandra` will use the old signatures, and
will be the simple wrappers of `cassandra` classes.

But before I do all this, I will move `CassandraConsistencyConfig` to
that `cassandra` package, because it is a part of new API.

To summarize:

`com.evolutiongaming.kafka.journal.cassandra` package in `cassandra`
project - will contain new common cassandra API used both by journaller
and snapshotter,

`com.evolutiongaming.kafka.journal.eventual.cassandra` package in
`eventual-cassandra` project - will contain an API used by journaller
only, including the legacy wrappers needed to maintain a binary
compatiblity with a previous version.

**Tricky questions**

*Why cannot you copy `EventualCassandraConfig` to `cassandra` instead?*

Because it contains dependencies on journaller specific classes such as
`SchemaConfig` or `SegmentSize`.

3313 of 4052 relevant lines covered (81.76%)

1.36 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
1
83.78
 -2.7% journal/src/main/scala/com/evolutiongaming/kafka/journal/HeadInfo.scala
6
90.83
 -2.75% eventual-cassandra/src/main/scala/com/evolutiongaming/kafka/journal/eventual/cassandra/ReplicatedCassandra.scala
Jobs
ID Job ID Ran Files Coverage
1 Scala 2.13.12 - 8249405725.1 191
81.76
Source Files on build 8249405725