From issues-return-31586-archive-asf-public=cust-asf.ponee.io@carbondata.apache.org Thu Jan 18 08:18:32 2018 Return-Path: X-Original-To: archive-asf-public@eu.ponee.io Delivered-To: archive-asf-public@eu.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by mx-eu-01.ponee.io (Postfix) with ESMTP id DEFE4180654 for ; Thu, 18 Jan 2018 08:18:32 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id CEA6D160C36; Thu, 18 Jan 2018 07:18:32 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 43CB6160C2B for ; Thu, 18 Jan 2018 08:18:32 +0100 (CET) Received: (qmail 81951 invoked by uid 500); 18 Jan 2018 07:18:31 -0000 Mailing-List: contact issues-help@carbondata.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@carbondata.apache.org Delivered-To: mailing list issues@carbondata.apache.org Received: (qmail 81942 invoked by uid 99); 18 Jan 2018 07:18:31 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 18 Jan 2018 07:18:31 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 42DA0E17C7; Thu, 18 Jan 2018 07:18:31 +0000 (UTC) From: mohammadshahidkhan To: issues@carbondata.apache.org Reply-To: issues@carbondata.apache.org Message-ID: Subject: [GitHub] carbondata pull request #1828: [CARBONDATA-2049] CarbonCleanFilesCommand tab... Content-Type: text/plain Date: Thu, 18 Jan 2018 07:18:31 +0000 (UTC) GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/carbondata/pull/1828 [CARBONDATA-2049] CarbonCleanFilesCommand table path problem **Problem:** In CarbonCleanFilesCommand datbaseLocation is being passed instead of the tablePath in case of forceclean. And in case of cleanGarbageData, storeLocation is being passed instead of the tablePath. - [X] Any interfaces changed? None - [X] Any backward compatibility impacted? None - [X] Document update required? None - [X] Testing done Please provide details on - Whether new unit test cases have been added or why no new tests are required? - How it is tested? Please attach test report. - Is it a performance related change? Please attach the performance test report. - Any additional information to help reviewers in testing this change. Corrected test case - [X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA. NA You can merge this pull request into a Git repository by running: $ git pull https://github.com/mohammadshahidkhan/incubator-carbondata cleancommand_fix Alternatively you can review and apply these changes as the patch at: https://github.com/apache/carbondata/pull/1828.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1828 ---- commit 8bfeb75b9af0de7d02f94a593c283d6d99df6f92 Author: mohammadshahidkhan Date: 2018-01-16T06:19:54Z [CARBONDATA-2049] CarbonCleanFilesCommand table path problem ---- ---