[GitHub] [carbondata] akashrn5 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
akashrn5 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546519558 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVManager.java ## @@ -114,7 +118,16 @@ public boolean hasSchemaOnTable(CarbonTable

[GitHub] [carbondata] akashrn5 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
akashrn5 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546519558 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVManager.java ## @@ -114,7 +118,16 @@ public boolean hasSchemaOnTable(CarbonTable

[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #4059: [CARBONDATA-4092] Fix concurrent issues in delete segment API's and MV flow

2020-12-20 Thread GitBox
Indhumathi27 commented on a change in pull request #4059: URL: https://github.com/apache/carbondata/pull/4059#discussion_r546534354 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVProvider.java ## @@ -540,7 +540,7 @@ private void

[GitHub] [carbondata] akashrn5 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
akashrn5 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546520134 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVManager.java ## @@ -234,7 +247,16 @@ public static String

[GitHub] [carbondata] akashrn5 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
akashrn5 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546520134 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVManager.java ## @@ -234,7 +247,16 @@ public static String

[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
Indhumathi27 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546528743 ## File path: core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java ## @@ -204,6 +205,17 @@ public

[GitHub] [carbondata] ajantha-bhat commented on a change in pull request #4059: [CARBONDATA-4092] Fix concurrent issues in delete segment API's and MV flow

2020-12-20 Thread GitBox
ajantha-bhat commented on a change in pull request #4059: URL: https://github.com/apache/carbondata/pull/4059#discussion_r546537345 ## File path: core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java ## @@ -445,38 +445,28 @@ private static

[GitHub] [carbondata] ajantha-bhat commented on a change in pull request #4059: [CARBONDATA-4092] Fix concurrent issues in delete segment API's and MV flow

2020-12-20 Thread GitBox
ajantha-bhat commented on a change in pull request #4059: URL: https://github.com/apache/carbondata/pull/4059#discussion_r546538696 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVProvider.java ## @@ -540,7 +540,7 @@ private void

[GitHub] [carbondata] ajantha-bhat commented on a change in pull request #4059: [CARBONDATA-4092] Fix concurrent issues in delete segment API's and MV flow

2020-12-20 Thread GitBox
ajantha-bhat commented on a change in pull request #4059: URL: https://github.com/apache/carbondata/pull/4059#discussion_r546538696 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVProvider.java ## @@ -540,7 +540,7 @@ private void

[GitHub] [carbondata] ajantha-bhat commented on a change in pull request #4059: [CARBONDATA-4092] Fix concurrent issues in delete segment API's and MV flow

2020-12-20 Thread GitBox
ajantha-bhat commented on a change in pull request #4059: URL: https://github.com/apache/carbondata/pull/4059#discussion_r546530685 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVProvider.java ## @@ -540,7 +540,7 @@ private void

[GitHub] [carbondata] akashrn5 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
akashrn5 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546522529 ## File path: integration/spark/src/main/scala/org/apache/spark/sql/optimizer/MVRewriteRule.scala ## @@ -183,7 +184,7 @@ class

[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
Indhumathi27 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546525745 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVManager.java ## @@ -234,7 +247,16 @@ public static String

[GitHub] [carbondata] akashrn5 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
akashrn5 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546520998 ## File path: integration/spark/src/main/scala/org/apache/carbondata/view/MVManagerInSpark.scala ## @@ -31,10 +31,10 @@ class

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
CarbonDataQA2 commented on pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#issuecomment-748824686 Build Failed with Spark 2.3.4, Please check CI http://121.244.95.60:12444/job/ApacheCarbonPRBuilder2.3/5218/

[GitHub] [carbondata] akashrn5 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
akashrn5 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546516920 ## File path: core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentStatusManager.java ## @@ -204,6 +205,17 @@ public

[GitHub] [carbondata] akashrn5 commented on a change in pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
akashrn5 commented on a change in pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#discussion_r546518847 ## File path: core/src/main/java/org/apache/carbondata/core/view/MVProvider.java ## @@ -364,18 +366,37 @@ private static boolean

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4060: [CARBONDATA-4093] Added logs for MV and method to verify if mv is in Sync during query

2020-12-20 Thread GitBox
CarbonDataQA2 commented on pull request #4060: URL: https://github.com/apache/carbondata/pull/4060#issuecomment-748824968 Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12444/job/ApacheCarbon_PR_Builder_2.4.5/3457/