You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2016/12/27 14:16:24 UTC
[08/12] kylin git commit: bug fix
bug fix
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/9f2d1145
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/9f2d1145
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/9f2d1145
Branch: refs/heads/mhb-master
Commit: 9f2d1145640be3dd47b90fcb1b64d796f87b5fbd
Parents: e14aadf
Author: Hongbin Ma <ma...@apache.org>
Authored: Sun Dec 25 14:51:08 2016 +0800
Committer: Hongbin Ma <ma...@apache.org>
Committed: Tue Dec 27 22:15:44 2016 +0800
----------------------------------------------------------------------
.../org/apache/kylin/query/enumerator/OLAPEnumerator.java | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/9f2d1145/query/src/main/java/org/apache/kylin/query/enumerator/OLAPEnumerator.java
----------------------------------------------------------------------
diff --git a/query/src/main/java/org/apache/kylin/query/enumerator/OLAPEnumerator.java b/query/src/main/java/org/apache/kylin/query/enumerator/OLAPEnumerator.java
index 408d227..e4fc937 100644
--- a/query/src/main/java/org/apache/kylin/query/enumerator/OLAPEnumerator.java
+++ b/query/src/main/java/org/apache/kylin/query/enumerator/OLAPEnumerator.java
@@ -76,14 +76,15 @@ public class OLAPEnumerator implements Enumerator<Object[]> {
}
convertCurrentRow(tuple);
return true;
- } finally {
+ } catch (Exception e) {
try {
if (cursor != null) {
cursor.close();
}
- } catch (Exception e) {
- logger.info("Error when closing cursor, ignore it", e);
+ } catch (Exception ee) {
+ logger.info("Error when closing cursor, ignore it", ee);
}
+ throw e;
}
}