You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@atlas.apache.org by sh...@apache.org on 2016/02/17 11:59:23 UTC

incubator-atlas git commit: ATLAS-457 Upgrade to 0.9 version of Kafka dependency (yhemanth via shwethags)

Repository: incubator-atlas
Updated Branches:
  refs/heads/master e06b1ee29 -> ddb14b54c


ATLAS-457 Upgrade to 0.9 version of Kafka dependency (yhemanth via shwethags)


Project: http://git-wip-us.apache.org/repos/asf/incubator-atlas/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-atlas/commit/ddb14b54
Tree: http://git-wip-us.apache.org/repos/asf/incubator-atlas/tree/ddb14b54
Diff: http://git-wip-us.apache.org/repos/asf/incubator-atlas/diff/ddb14b54

Branch: refs/heads/master
Commit: ddb14b54cd7994a1bc5afe0155a95ef10f101124
Parents: e06b1ee
Author: Shwetha GS <ss...@hortonworks.com>
Authored: Wed Feb 17 16:29:16 2016 +0530
Committer: Shwetha GS <ss...@hortonworks.com>
Committed: Wed Feb 17 16:29:16 2016 +0530

----------------------------------------------------------------------
 .../java/org/apache/atlas/kafka/KafkaNotification.java    |  6 ++++--
 pom.xml                                                   |  2 +-
 release-log.txt                                           |  1 +
 .../org/apache/atlas/web/security/BaseSecurityTest.java   | 10 ++++++++++
 4 files changed, 16 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-atlas/blob/ddb14b54/notification/src/main/java/org/apache/atlas/kafka/KafkaNotification.java
----------------------------------------------------------------------
diff --git a/notification/src/main/java/org/apache/atlas/kafka/KafkaNotification.java b/notification/src/main/java/org/apache/atlas/kafka/KafkaNotification.java
index 7c91923..2701039 100644
--- a/notification/src/main/java/org/apache/atlas/kafka/KafkaNotification.java
+++ b/notification/src/main/java/org/apache/atlas/kafka/KafkaNotification.java
@@ -43,6 +43,7 @@ import org.apache.zookeeper.server.ServerCnxnFactory;
 import org.apache.zookeeper.server.ZooKeeperServer;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
+import scala.Option;
 
 import java.io.File;
 import java.io.IOException;
@@ -117,7 +118,7 @@ public class KafkaNotification extends AbstractNotification implements Service {
                 "org.apache.kafka.common.serialization.StringDeserializer");
         properties.put(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG,
                 "org.apache.kafka.common.serialization.StringDeserializer");
-        properties.put(ConsumerConfig.PARTITION_ASSIGNMENT_STRATEGY, "roundrobin");
+        properties.put(ConsumerConfig.PARTITION_ASSIGNMENT_STRATEGY_CONFIG, "roundrobin");
         properties.put(ConsumerConfig.AUTO_OFFSET_RESET_CONFIG, "smallest");
     }
 
@@ -303,7 +304,8 @@ public class KafkaNotification extends AbstractNotification implements Service {
         brokerConfig.setProperty("log.dirs", constructDir("kafka").getAbsolutePath());
         brokerConfig.setProperty("log.flush.interval.messages", String.valueOf(1));
 
-        kafkaServer = new KafkaServer(new KafkaConfig(brokerConfig), new SystemTime());
+        kafkaServer = new KafkaServer(KafkaConfig.fromProps(brokerConfig), new SystemTime(),
+                Option.apply(this.getClass().getName()));
         kafkaServer.startup();
         LOG.debug("Embedded kafka server started with broker config {}", brokerConfig);
     }

http://git-wip-us.apache.org/repos/asf/incubator-atlas/blob/ddb14b54/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index c74e31f..0837817 100755
--- a/pom.xml
+++ b/pom.xml
@@ -342,7 +342,7 @@
         <hadoop.version>2.7.0</hadoop.version>
         <hbase.version>1.1.2</hbase.version>
         <solr.version>5.1.0</solr.version>
-        <kafka.version>0.8.2.0</kafka.version>
+        <kafka.version>0.9.0.0</kafka.version>
         <!-- scala versions -->
         <scala.version>2.10.4</scala.version>
         <scala.binary.version>2.10</scala.binary.version>

http://git-wip-us.apache.org/repos/asf/incubator-atlas/blob/ddb14b54/release-log.txt
----------------------------------------------------------------------
diff --git a/release-log.txt b/release-log.txt
index e740c93..8a7d2fd 100644
--- a/release-log.txt
+++ b/release-log.txt
@@ -7,6 +7,7 @@ ATLAS-409 Atlas will not import avro tables with schema read from a file (dosset
 ATLAS-379 Create sqoop and falcon metadata addons (venkatnrangan,bvellanki,sowmyaramesh via shwethags)
 
 ALL CHANGES:
+ATLAS-457 Upgrade to 0.9 version of Kafka dependency (yhemanth via shwethags)
 ATLAS-398 Delete trait that exists but not linked to entity results in "400 Bad request". It should result "404 not found" (ndjouhr via shwethags)
 ATLAS-372 Expose entity deletion through REST API (dkantor via shwethags)
 ATLAS-452 Exceptions while running HiveHookIT#testAlterTableRename (shwethags)

http://git-wip-us.apache.org/repos/asf/incubator-atlas/blob/ddb14b54/webapp/src/test/java/org/apache/atlas/web/security/BaseSecurityTest.java
----------------------------------------------------------------------
diff --git a/webapp/src/test/java/org/apache/atlas/web/security/BaseSecurityTest.java b/webapp/src/test/java/org/apache/atlas/web/security/BaseSecurityTest.java
index 8909e89..e010537 100644
--- a/webapp/src/test/java/org/apache/atlas/web/security/BaseSecurityTest.java
+++ b/webapp/src/test/java/org/apache/atlas/web/security/BaseSecurityTest.java
@@ -100,6 +100,16 @@ public class BaseSecurityTest {
     protected void bindJVMtoJAASFile(File jaasFile) {
         String path = jaasFile.getAbsolutePath();
         System.setProperty(Environment.JAAS_CONF_KEY, path);
+        disableZookeeperSecurity();
+    }
+
+    /* We only want Atlas to work in secure mode for the tests
+     * for otherwise a lot more configuration is required to
+     * make other components like Kafka run in secure mode.
+     */
+    private void disableZookeeperSecurity() {
+        System.setProperty("zookeeper.sasl.client", "false");
+        System.setProperty("zookeeper.sasl.clientconfig", "");
     }
 
     protected File createKeytab(MiniKdc kdc, File kdcWorkDir, String principal, String filename) throws Exception {