incubator-gora-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Patricio Echagüe <patric...@gmail.com>
Subject Re: Comments about Hector usage
Date Mon, 16 Jan 2012 15:57:34 GMT
thanks Lewis. I'll definitely try to submit a patch.

Sent from my Android
On Jan 16, 2012 3:04 AM, "Lewis John Mcgibbney" <lewis.mcgibbney@gmail.com>
wrote:

> Hi Patricio,
>
> This is great feedback thanks very much. To log and deal with the points
> you raise I've opened some tickets on our Jira issue tracker. The parent
> issue is here [1]. At this stage, when I get a bit of time I'll try my best
> to have a look and get some patches submitted, however if you are available
> or get a bit of spare time and wish to submit any work then please feel
> free to add your implementations, anything would be greatly appreciated.
>
> Thanks again for reporting this.
>
> Best
>
> Lewis
>
> [1] https://issues.apache.org/jira/browse/GORA-78
>
> 2012/1/15 Patricio Echagüe <patricioe@gmail.com>
>
> > - CassandraClient#checkKeyspace() :
> >
> >    Not sure if this is intentional but ideally during a keyspace creation
> > you may want to block until the whole cluster converge to the new
> keyspace.
> > You can achieve that by adding an extra boolean parameter during the
> method
> > call: -> this.cluster.addKeyspace(keyspaceDefinition, true);
> >    Hector will block until all nodes in the cluster agree on the new
> > keyspace.
> >
> > - Is there a way in Gora to define the Consistency used for read and
> write
> > operations? I see the CassandraClient (Hector) being used with default
> > values.
> >
> > - CassandraStore#addOrUpdateField contains a switch statement to evaluate
> > different types of column values but in the end the cassandraClient
> object
> > end up converting to string anyway. I think, the InferringTypeSerializer
> > can be used instead in order to take advantage of when the value is
> already
> > of type ByteBuffer.
> >
> >
> > I know you guys are just starting to work on this. These are minor
> comments
> > and it looks like Hector is being used in the right way.
> >
> > No more comments so far.
> >
> > Cheers
> > Patricio
> >
>
>
>
> --
> *Lewis*
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message