You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by wy...@apache.org on 2018/12/15 17:39:48 UTC

asterixdb git commit: [ASTERIXDB-2492][TXN] Skip filter update on undo

Repository: asterixdb
Updated Branches:
  refs/heads/master 440e3a56b -> 81f3e7593


[ASTERIXDB-2492][TXN] Skip filter update on undo

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
Filter values should not be undone on UPDATE as the tuple
does not contain the filtered field.

Change-Id: I1b9c5d0350e1df94369e6733069dcf5af11a78b6
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3088
Sonar-Qube: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
Contrib: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Murtadha Hubail <mh...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/asterixdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/asterixdb/commit/81f3e759
Tree: http://git-wip-us.apache.org/repos/asf/asterixdb/tree/81f3e759
Diff: http://git-wip-us.apache.org/repos/asf/asterixdb/diff/81f3e759

Branch: refs/heads/master
Commit: 81f3e7593ad9d4170aa3bd47c5c6aae4183873c5
Parents: 440e3a5
Author: Wail Alkowaileet <wa...@gmail.com>
Authored: Thu Dec 13 17:25:43 2018 -0800
Committer: Wail Alkowaileet <wa...@gmail.com>
Committed: Sat Dec 15 09:38:47 2018 -0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/asterix/app/nc/RecoveryManager.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/81f3e759/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java
----------------------------------------------------------------------
diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java
index 946815f..3fce4d8 100644
--- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java
+++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java
@@ -75,7 +75,6 @@ import org.apache.hyracks.api.lifecycle.ILifeCycleComponent;
 import org.apache.hyracks.storage.am.common.impls.NoOpIndexAccessParameters;
 import org.apache.hyracks.storage.am.lsm.common.api.ILSMComponentId;
 import org.apache.hyracks.storage.am.lsm.common.api.ILSMComponentId.IdCompareResult;
-import org.apache.hyracks.storage.am.lsm.common.api.ILSMComponentIdGenerator;
 import org.apache.hyracks.storage.am.lsm.common.api.ILSMDiskComponent;
 import org.apache.hyracks.storage.am.lsm.common.api.ILSMIOOperation;
 import org.apache.hyracks.storage.am.lsm.common.api.ILSMIOOperation.LSMIOOperationStatus;
@@ -752,6 +751,8 @@ public class RecoveryManager implements IRecoveryManager, ILifeCycleComponent {
             ILSMIndex index =
                     (ILSMIndex) datasetLifecycleManager.getIndex(logRecord.getDatasetId(), logRecord.getResourceId());
             ILSMIndexAccessor indexAccessor = index.createAccessor(NoOpIndexAccessParameters.INSTANCE);
+            ILSMIndexOperationContext opCtx = indexAccessor.getOpContext();
+            opCtx.setFilterSkip(true);
             try {
                 switch (logRecord.getNewOp()) {
                     case AbstractIndexModificationOperationCallback.INSERT_BYTE: