Return-Path: X-Original-To: apmail-ambari-dev-archive@www.apache.org Delivered-To: apmail-ambari-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 258BE10C40 for ; Thu, 22 Oct 2015 17:30:28 +0000 (UTC) Received: (qmail 38933 invoked by uid 500); 22 Oct 2015 17:30:06 -0000 Delivered-To: apmail-ambari-dev-archive@ambari.apache.org Received: (qmail 38900 invoked by uid 500); 22 Oct 2015 17:30:06 -0000 Mailing-List: contact dev-help@ambari.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ambari.apache.org Delivered-To: mailing list dev@ambari.apache.org Received: (qmail 38882 invoked by uid 99); 22 Oct 2015 17:30:05 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 22 Oct 2015 17:30:05 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 06DEA278DE7; Thu, 22 Oct 2015 17:30:05 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============4630735398002816032==" MIME-Version: 1.0 Subject: Re: Review Request 39448: Upgrade path on ORA 1.6.1 -> 2.0.1 -> 2.1.1 From: "Dmytro Sen" To: "Myroslav Papirkovskyy" , "Sid Wagle" Cc: "Dmytro Sen" , "Ambari" Date: Thu, 22 Oct 2015 17:30:05 -0000 Message-ID: <20151022173005.1666.21060@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Dmytro Sen" X-ReviewGroup: Ambari X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/39448/ X-Sender: "Dmytro Sen" References: <20151019183836.32263.43658@reviews.apache.org> In-Reply-To: <20151019183836.32263.43658@reviews.apache.org> Reply-To: "Dmytro Sen" X-ReviewRequest-Repository: ambari --===============4630735398002816032== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39448/ ----------------------------------------------------------- (Updated Окт. 22, 2015, 5:30 п.п.) Review request for Ambari, Myroslav Papirkovskyy and Sid Wagle. Bugs: AMBARI-13478 https://issues.apache.org/jira/browse/AMBARI-13478 Repository: ambari Description ------- We ignore case on trying find if FK exists but use the user supplied fk name in ALTER query. This method should return the fk name found if at all. Diffs (updated) ----- ambari-server/src/main/java/org/apache/ambari/server/orm/DBAccessorImpl.java cd47252 ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java 12a1f44 ambari-server/src/test/java/org/apache/ambari/server/orm/DBAccessorImplTest.java 05856b3 Diff: https://reviews.apache.org/r/39448/diff/ Testing ------- Unit tests passed Thanks, Dmytro Sen --===============4630735398002816032==--