IGNITE-10856 cassandra-store seems to be broken by incorrect guava version

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

IGNITE-10856 cassandra-store seems to be broken by incorrect guava version

Saikat Maitra
Hi,

I have raised a PR for the below issue.

IGNITE-10856 cassandra-store seems to be broken by incorrect guava version

PR - https://github.com/apache/ignite/pull/5780/

Please take a look and share feedback.

Regards,
Saikat
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-10856 cassandra-store seems to be broken by incorrect guava version

Dmitriy Pavlov-3
Hi Saikat,

Fix itself is ok. But from the top of my head, I don't know how can we
auto-test this scenario in the future. Do you have an idea?

Sincerely,
Dmitriy Pavlov

ср, 9 янв. 2019 г. в 07:08, Saikat Maitra <[hidden email]>:

> Hi,
>
> I have raised a PR for the below issue.
>
> IGNITE-10856 cassandra-store seems to be broken by incorrect guava version
>
> PR - https://github.com/apache/ignite/pull/5780/
>
> Please take a look and share feedback.
>
> Regards,
> Saikat
>
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-10856 cassandra-store seems to be broken by incorrect guava version

Saikat Maitra
Hi Dmitriy,

Ilya has created following jira issue to address the problem

https://issues.apache.org/jira/browse/IGNITE-10879

We will need to create a TestSuite for cassandra store and also include
this following test which will help in validation

IgnitePersistentStoreTest

Regards,
Saikat


On Wed, Jan 9, 2019 at 1:03 AM Dmitriy Pavlov <[hidden email]> wrote:

> Hi Saikat,
>
> Fix itself is ok. But from the top of my head, I don't know how can we
> auto-test this scenario in the future. Do you have an idea?
>
> Sincerely,
> Dmitriy Pavlov
>
> ср, 9 янв. 2019 г. в 07:08, Saikat Maitra <[hidden email]>:
>
> > Hi,
> >
> > I have raised a PR for the below issue.
> >
> > IGNITE-10856 cassandra-store seems to be broken by incorrect guava
> version
> >
> > PR - https://github.com/apache/ignite/pull/5780/
> >
> > Please take a look and share feedback.
> >
> > Regards,
> > Saikat
> >
>