Return-Path: X-Original-To: apmail-hive-dev-archive@www.apache.org Delivered-To: apmail-hive-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 13863E2BE for ; Fri, 1 Feb 2013 06:31:17 +0000 (UTC) Received: (qmail 55953 invoked by uid 500); 1 Feb 2013 06:31:16 -0000 Delivered-To: apmail-hive-dev-archive@hive.apache.org Received: (qmail 55578 invoked by uid 500); 1 Feb 2013 06:31:15 -0000 Mailing-List: contact dev-help@hive.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hive.apache.org Delivered-To: mailing list dev@hive.apache.org Received: (qmail 55525 invoked by uid 500); 1 Feb 2013 06:31:14 -0000 Delivered-To: apmail-hadoop-hive-dev@hadoop.apache.org Received: (qmail 55516 invoked by uid 99); 1 Feb 2013 06:31:14 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 01 Feb 2013 06:31:14 +0000 Date: Fri, 1 Feb 2013 06:31:14 +0000 (UTC) From: "Kevin Wilfong (JIRA)" To: hive-dev@hadoop.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (HIVE-3970) Clean up/fix PartitionNameWhitelistPreEventListener MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/HIVE-3970?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Wilfong updated HIVE-3970: -------------------------------- Status: Patch Available (was: Open) > Clean up/fix PartitionNameWhitelistPreEventListener > --------------------------------------------------- > > Key: HIVE-3970 > URL: https://issues.apache.org/jira/browse/HIVE-3970 > Project: Hive > Issue Type: Improvement > Components: Metastore > Affects Versions: 0.11.0 > Reporter: Kevin Wilfong > Assignee: Kevin Wilfong > Attachments: HIVE-3970.1.patch.txt > > > There are a number of issues and things which can be cleaned up related to PartitionNameWhitelistPreEventListener. > * It's an event listener, but it really doesn't need to be given that the regex whitelist is configurable, it could just be a utility method. > * It's not run when a partition is renamed, so partitions with invalid characters can be created in this way. > * There's no easy way to check if a partition contains invalid characters before creating it and seeing if it fails. > Most importantly, when a dynamic partition contains an invalid character, the directory for this partition is created, and the data is moved into it, but the partition fails to be created leaving an orphan directory. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira