cassandra-pr mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From johnyannj <...@git.apache.org>
Subject [GitHub] cassandra pull request #137: CASSANDRA-13363/CASSANDRA-12435 fix index flag ...
Date Wed, 09 Aug 2017 03:59:11 GMT
GitHub user johnyannj opened a pull request:

    https://github.com/apache/cassandra/pull/137

    CASSANDRA-13363/CASSANDRA-12435 fix index flag before not be consistent with follow data
content

    fix CASSANDRA-13363/CASSANDRA-12435
    
    when the command will be executed by local node and target node.
    the command.index will be reload by local thread.
    
    when code reach indexFlag(command.index.isPresent())  is false
    but reach if (command.index.isPresent()) , it is changed to true, because the "command.index"
has been loaded by local execute thread.
    
    the command.index can be reload by target node, so it's serialize is optional。 but the
flag must be consistent with follow data content.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/johnyannj/cassandra johnyannj-patch-fix-13363

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cassandra/pull/137.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 #137
    
----
commit 5050ff51b1f62fe7a678be01f79ff962a0fd6caa
Author: zhaoyan <zhaoyan@zhaoyanblog.com>
Date:   2017-08-09T02:35:46Z

    Update ReadCommand.java
    
    fix CASSANDRA-13363/CASSANDRA-12435
    
    when the command will be executed by local node and target node.
    the command.index will be reload by local thread.
    
    when code reach indexFlag(command.index.isPresent())  is false
    but reach if (command.index.isPresent()) , it is changed to true, because the "command.index"
has been loaded by local execute thread.
    
    the command.index can be reload by target node, so it's serialize is optional。 but the
flag must be consistent with follow data content.

----


---
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.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org


Mime
View raw message