Return-Path: X-Original-To: apmail-tajo-dev-archive@minotaur.apache.org Delivered-To: apmail-tajo-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 6E0841044F for ; Sun, 29 Dec 2013 10:54:16 +0000 (UTC) Received: (qmail 17977 invoked by uid 500); 29 Dec 2013 10:54:15 -0000 Delivered-To: apmail-tajo-dev-archive@tajo.apache.org Received: (qmail 17946 invoked by uid 500); 29 Dec 2013 10:54:15 -0000 Mailing-List: contact dev-help@tajo.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@tajo.incubator.apache.org Delivered-To: mailing list dev@tajo.incubator.apache.org Received: (qmail 17937 invoked by uid 99); 29 Dec 2013 10:54:12 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 29 Dec 2013 10:54:12 +0000 X-ASF-Spam-Status: No, hits=-2000.5 required=5.0 tests=ALL_TRUSTED,RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO mail.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with SMTP; Sun, 29 Dec 2013 10:54:11 +0000 Received: (qmail 17811 invoked by uid 99); 29 Dec 2013 10:53:50 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 29 Dec 2013 10:53:50 +0000 Date: Sun, 29 Dec 2013 10:53:50 +0000 (UTC) From: "Hyunsik Choi (JIRA)" To: dev@tajo.incubator.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Comment Edited] (TAJO-455) Throw PlanningException when Creating table with Partition exception COLUMN MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/TAJO-455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13858314#comment-13858314 ] Hyunsik Choi edited comment on TAJO-455 at 12/29/13 10:52 AM: -------------------------------------------------------------- I like this fix. By the way, wrong format strings are used in this patch. 792 in LogicalPlanner will cause java.util.IllegalFormatConversionException because the format string is '%d' but PartitionType is an enum type. 357 in GlobalPlanner will be the same to the above. In addition, 792 and 835 lines width in LogicalPlanner exceed 120 column. Could you reformat them? Thanks! was (Author: hyunsik): I like this fix. But, wrong format strings are used in this patch. 792 in LogicalPlanner will cause java.util.IllegalFormatConversionException because the format string is '%d' but PartitionType is an enum type. 357 in GlobalPlanner will be the same to the above. In addition, 792 and 835 lines width in LogicalPlanner exceed 120 column. Could you reformat them? Thanks! > Throw PlanningException when Creating table with Partition exception COLUMN > --------------------------------------------------------------------------- > > Key: TAJO-455 > URL: https://issues.apache.org/jira/browse/TAJO-455 > Project: Tajo > Issue Type: Improvement > Components: planner/optimizer > Reporter: DaeMyung Kang > Assignee: DaeMyung Kang > Attachments: TAJO-455.patch > > > currently Tajo just support COLUMN Partitoner > but Tajo accept other Partitioner (HASH, LIST, RANGE) > so users can create table with unsupported Partitioner. > but tajo will throws NULL Pointer Exception. > so. until supporting these Partitioner(HASH, LIST, RANGE) > I think it is better blcok to create table with Partitioner except COLUMN. > but, they will be supported soon in future. > I don't remove codes and tests. > and just block them. > 1. throw exception when creating table with Partitioner except(COLUMN) in logicalPlanner. > 2. just remove @Test clauese from some test cases. > 3. add testcase to check failing create table with hash, list, range partitioner > I might think this patch give more good experience to users. -- This message was sent by Atlassian JIRA (v6.1.5#6160)