You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@carbondata.apache.org by ra...@apache.org on 2017/04/21 07:27:07 UTC
[1/2] incubator-carbondata git commit: [CARBONDATA-881] Load status
is successful even though system is fail to write status into tablestatus
file
Repository: incubator-carbondata
Updated Branches:
refs/heads/master 5c69c4918 -> f59f5ae81
[CARBONDATA-881] Load status is successful even though system is fail to write status into tablestatus file
Project: http://git-wip-us.apache.org/repos/asf/incubator-carbondata/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-carbondata/commit/5e40f165
Tree: http://git-wip-us.apache.org/repos/asf/incubator-carbondata/tree/5e40f165
Diff: http://git-wip-us.apache.org/repos/asf/incubator-carbondata/diff/5e40f165
Branch: refs/heads/master
Commit: 5e40f165a3b252fa7e8602a1d67571bfb0fcfcbd
Parents: 5c69c49
Author: mohammadshahidkhan <mo...@gmail.com>
Authored: Thu Apr 6 19:14:50 2017 +0530
Committer: ravipesala <ra...@gmail.com>
Committed: Fri Apr 21 12:56:09 2017 +0530
----------------------------------------------------------------------
.../carbondata/core/statusmanager/SegmentStatusManager.java | 3 +++
.../apache/carbondata/processing/merger/CarbonDataMergerUtil.java | 1 +
2 files changed, 4 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-carbondata/blob/5e40f165/core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java b/core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java
index c2c41e5..63318fe 100644
--- a/core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java
+++ b/core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java
@@ -292,6 +292,7 @@ public class SegmentStatusManager {
}
} catch (IOException e) {
LOG.error("IOException" + e.getMessage());
+ throw e;
} finally {
CarbonLockUtil.fileUnlock(carbonTableStatusLock, LockUsage.TABLE_STATUS_LOCK);
CarbonLockUtil.fileUnlock(carbonDeleteSegmentLock, LockUsage.DELETE_SEGMENT_LOCK);
@@ -377,6 +378,7 @@ public class SegmentStatusManager {
}
} catch (IOException e) {
LOG.error("Error message: " + "IOException" + e.getMessage());
+ throw e;
} finally {
CarbonLockUtil.fileUnlock(carbonTableStatusLock, LockUsage.TABLE_STATUS_LOCK);
CarbonLockUtil.fileUnlock(carbonDeleteSegmentLock, LockUsage.DELETE_SEGMENT_LOCK);
@@ -410,6 +412,7 @@ public class SegmentStatusManager {
brWriter.write(metadataInstance);
} catch (IOException ioe) {
LOG.error("Error message: " + ioe.getLocalizedMessage());
+ throw ioe;
} finally {
if (null != brWriter) {
brWriter.flush();
http://git-wip-us.apache.org/repos/asf/incubator-carbondata/blob/5e40f165/processing/src/main/java/org/apache/carbondata/processing/merger/CarbonDataMergerUtil.java
----------------------------------------------------------------------
diff --git a/processing/src/main/java/org/apache/carbondata/processing/merger/CarbonDataMergerUtil.java b/processing/src/main/java/org/apache/carbondata/processing/merger/CarbonDataMergerUtil.java
index 2414993..65ec4fc 100644
--- a/processing/src/main/java/org/apache/carbondata/processing/merger/CarbonDataMergerUtil.java
+++ b/processing/src/main/java/org/apache/carbondata/processing/merger/CarbonDataMergerUtil.java
@@ -383,6 +383,7 @@ public final class CarbonDataMergerUtil {
tableStatusUpdationStatus = true;
} catch (IOException e) {
LOGGER.error("Error while writing metadata");
+ tableStatusUpdationStatus = false;
}
} else {
LOGGER.error(
[2/2] incubator-carbondata git commit: [CARBONDATA-881] Load status
is successful even though system is fail to write status into tablestatus
file. This closes #755
Posted by ra...@apache.org.
[CARBONDATA-881] Load status is successful even though system is fail to write status into tablestatus file. This closes #755
Project: http://git-wip-us.apache.org/repos/asf/incubator-carbondata/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-carbondata/commit/f59f5ae8
Tree: http://git-wip-us.apache.org/repos/asf/incubator-carbondata/tree/f59f5ae8
Diff: http://git-wip-us.apache.org/repos/asf/incubator-carbondata/diff/f59f5ae8
Branch: refs/heads/master
Commit: f59f5ae8103a65a3daa49c12bb23c5c28f538c2b
Parents: 5c69c49 5e40f16
Author: ravipesala <ra...@gmail.com>
Authored: Fri Apr 21 12:56:47 2017 +0530
Committer: ravipesala <ra...@gmail.com>
Committed: Fri Apr 21 12:56:47 2017 +0530
----------------------------------------------------------------------
.../carbondata/core/statusmanager/SegmentStatusManager.java | 3 +++
.../apache/carbondata/processing/merger/CarbonDataMergerUtil.java | 1 +
2 files changed, 4 insertions(+)
----------------------------------------------------------------------