From c0a43d436fdd9293bf10e4b874d1002441c056b1 Mon Sep 17 00:00:00 2001 From: Kate Ting Date: Mon, 13 May 2013 00:46:06 -0400 Subject: [PATCH] SQOOP-1041: Sqoop2: Framework upgrade in JdbcRepository returns the outdated value (Jarek Jarcec Cecho via Kate Ting) --- .../java/org/apache/sqoop/repository/JdbcRepository.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java b/core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java index bc6af375..d42e34af 100644 --- a/core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java +++ b/core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java @@ -183,7 +183,12 @@ public Object doIt(Connection conn) { handler.registerFramework(mFramework, conn); return mFramework; } else { - upgradeFramework(mFramework); + // We're currently not serializing framework version into repository + // so let's just compare the structure to see if we need upgrade. + if(!mFramework.equals(result)) { + upgradeFramework(mFramework); + return mFramework; + } return result; } }