Apache CarbonData Dev Mailing List archive › Apache CarbonData JIRA issues

[GitHub] carbondata pull request #1847: [CARBONDATA-2064] Add compaction listener

Classic

List

Threaded

44 messages Options

123

qiuchenjian-2

[GitHub] carbondata issue #1847: [WIP][CARBONDATA-2064] Add compaction listener

Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1847

Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/3225/

---

qiuchenjian-2

[GitHub] carbondata issue #1847: [WIP][CARBONDATA-2064] Add compaction listener

In reply to this post by qiuchenjian-2

Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1847

Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1991/

---

qiuchenjian-2

[GitHub] carbondata issue #1847: [WIP][CARBONDATA-2064] Add compaction listener

In reply to this post by qiuchenjian-2

Github user ravipesala commented on the issue:

https://github.com/apache/carbondata/pull/1847

SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3184/

---

qiuchenjian-2

[GitHub] carbondata issue #1847: [WIP][CARBONDATA-2064] Add compaction listener

In reply to this post by qiuchenjian-2

Github user dhatchayani commented on the issue:

https://github.com/apache/carbondata/pull/1847

retest sdv please

---

qiuchenjian-2

[GitHub] carbondata issue #1847: [WIP][CARBONDATA-2064] Add compaction listener

In reply to this post by qiuchenjian-2

Github user ravipesala commented on the issue:

https://github.com/apache/carbondata/pull/1847

SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/3198/

---

qiuchenjian-2

[GitHub] carbondata pull request #1847: [WIP][CARBONDATA-2064] Add compaction listene...

In reply to this post by qiuchenjian-2

Github user ravipesala commented on a diff in the pull request:

https://github.com/apache/carbondata/pull/1847#discussion_r164939374

--- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/datacompaction/CarbonIndexFileMergeTestCase.scala ---
@@ -1,199 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package org.apache.carbondata.spark.testsuite.datacompaction
-
-import org.apache.spark.sql.test.util.QueryTest
-import org.scalatest.{BeforeAndAfterAll, BeforeAndAfterEach}
-
-import org.apache.carbondata.core.constants.CarbonCommonConstants
-import org.apache.carbondata.core.datastore.filesystem.{CarbonFile, CarbonFileFilter}
-import org.apache.carbondata.core.datastore.impl.FileFactory
-import org.apache.carbondata.core.metadata.CarbonMetadata
-import org.apache.carbondata.core.util.CarbonProperties
-import org.apache.carbondata.core.util.path.CarbonTablePath
-
-class CarbonIndexFileMergeTestCase
--- End diff --

Don't remove the test case, use direct API to merge instead of using DDL

---

qiuchenjian-2