debop / hibernate-redis

hibernate 2nd level cache privder using redis
Apache License 2.0
357 stars 184 forks source link

Suggested dependencies for the readme #53

Closed ignaciolarranaga closed 8 years ago

ignaciolarranaga commented 8 years ago

Hi just gone through the configuration with hibernate5/spring and detected some probably missed dependencies. Don't know but commenting just in case.

<dependency>
    <groupId>org.redisson</groupId>
    <artifactId>redisson</artifactId>
    <version>${redisson.version}</version>
</dependency>
<dependency>
    <groupId>com.github.debop</groupId>
    <artifactId>hibernate-redis</artifactId>
    <version>${hibernate-redis.version}</version>
    <exclusions>
        <exclusion>
            <groupId>ch.qos.logback</groupId>
            <artifactId>logback-core</artifactId>
        </exclusion>
        <exclusion>
            <groupId>ch.qos.logback</groupId>
            <artifactId>logback-classic</artifactId>
        </exclusion>
    </exclusions>
</dependency>
<dependency>
    <groupId>de.ruedigermoeller</groupId>
    <artifactId>fst</artifactId>
    <version>${fst.version}</version>
</dependency>
<dependency>
    <groupId>org.xerial.snappy</groupId>
    <artifactId>snappy-java</artifactId>
    <version>${snappy-java.version}</version>
</dependency>
debop commented 8 years ago

OK. Thanks your sugestion.

debop commented 8 years ago

add optional depencencies information to README.md (2.0.2)

contentfree commented 7 years ago

Any chance of removing the logback dependency altogether? It causes maven to complain about SLF4J to complain about multiple bindings.

contentfree commented 7 years ago

Or at least mark it as a development / test dependency so that it's not included upstream.