He-Pin opened a new pull request, #425:
URL: https://github.com/apache/pekko-persistence-cassandra/pull/425
### Motivation
Compilation produces several warnings that should be resolved:
- "a type was inferred to be `Any`; this may indicate a programming error"
in `CassandraJournal.scala:898` and `CassandraSnapshotStore.scala:374`
- "match may not be exhaustive" in `CassandraSnapshotStore.scala:123`
- sbt lint warnings about unused keys `projectInfoVersion` (core, dseTest,
endToEndExample, root) and `previewPath` (docs/previewSite)
### Modification
- Add explicit `Array[AnyRef]` type annotation and `Long.box`/`Int.box` for
primitive values in `log.warning` `Array` arguments in both `CassandraJournal`
and `CassandraSnapshotStore`
- Add `@nowarn("msg=match may not be exhaustive")` on `loadNAsync` where the
`Seq()` and `+:` patterns are actually exhaustive for `immutable.Seq` but the
compiler does not recognize this
- Add `excludeLintKeys` for `projectInfoVersion` in `Common.scala` and
`previewPath` in `build.sbt`
### Result
Clean compilation with no warnings.
### Tests
- `sbt "clean; compile"` passes with no warnings
### References
None - compiler warning cleanup
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
