You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@usergrid.apache.org by yding37 <gi...@git.apache.org> on 2015/04/16 01:15:37 UTC
[GitHub] incubator-usergrid pull request: Add hector settings
GitHub user yding37 opened a pull request:
https://github.com/apache/incubator-usergrid/pull/225
Add hector settings
Enable setting the following values for hector:
- cassandraThriftSocketTimeout
- useSocketKeepalive
Enabled HectorLockManager
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/yding37/incubator-usergrid master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-usergrid/pull/225.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #225
----
commit cdaabe3afca13cb5ca72f51b98c062bc0610d154
Author: Yi Ding <yd...@gmail.com>
Date: 2015-04-15T23:13:15Z
add hector settings for timeout and keep alive. enable HectorLockManagerImpl
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-usergrid pull request: Add hector settings
Posted by snoopdave <gi...@git.apache.org>.
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/225#discussion_r28530186
--- Diff: stack/config/src/main/resources/usergrid-default.properties ---
@@ -62,6 +62,15 @@ cassandra.mutation.flushsize=2000
#Note that if this is deployed in a production cluster, the RF on the keyspace MUST be updated to use an odd number for it's replication Factor.
#Even numbers for RF can potentially case the locks to fail, via "split brain" when read at QUORUM on lock verification
cassandra.lock.keyspace=Locks
+#locking read & write policies
+cassandra.lock.readcl=QUORUM
--- End diff --
both of these should be local quorum by default
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-usergrid pull request: Add hector settings
Posted by yding37 <gi...@git.apache.org>.
Github user yding37 commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/225#issuecomment-93834240
I'm not too sure actually, although the version that's shipped with
usergrid has that setting.
On Thu, Apr 16, 2015 at 12:42 PM, David M. Johnson <notifications@github.com
> wrote:
> what version of Hector adds the new cassandraThriftSocketTimeout
> parameters?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/incubator-usergrid/pull/225#issuecomment-93823653>
> .
>
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-usergrid pull request: Add hector settings
Posted by GERey <gi...@git.apache.org>.
Github user GERey commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/225#issuecomment-130467710
Hello! If you could update this PR to address the conflicts I would be more than happy to accept the PR. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-usergrid pull request: Add hector settings
Posted by yding37 <gi...@git.apache.org>.
Github user yding37 commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/225#issuecomment-93835128
is this necessary for build purposes?
On Thu, Apr 16, 2015 at 1:37 PM, Yi Ding <yd...@gmail.com> wrote:
> I'm not too sure actually, although the version that's shipped with
> usergrid has that setting.
>
> On Thu, Apr 16, 2015 at 12:42 PM, David M. Johnson <
> notifications@github.com> wrote:
>
>> what version of Hector adds the new cassandraThriftSocketTimeout
>> parameters?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/apache/incubator-usergrid/pull/225#issuecomment-93823653>
>> .
>>
>
>
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-usergrid pull request: Add hector settings
Posted by snoopdave <gi...@git.apache.org>.
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/225#issuecomment-93823653
what version of Hector adds the new cassandraThriftSocketTimeout parameters?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---