[jira] [Updated] (ATLAS-4440) Upgrade Atlas's Kafka dependency to 2.8
[ https://issues.apache.org/jira/browse/ATLAS-4440?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-4440: --- Fix Version/s: 2.3.0 > Upgrade Atlas's Kafka dependency to 2.8 > --- > > Key: ATLAS-4440 > URL: https://issues.apache.org/jira/browse/ATLAS-4440 > Project: Atlas > Issue Type: Improvement > Components: atlas-core > Reporter: Viktor Somogyi-Vass > Assignee: Viktor Somogyi-Vass >Priority: Major > Fix For: 3.0.0, 2.3.0 > > Attachments: 0001-ATLAS-4440-Update-Kafka-dependency-to-2.8.1.patch > > -- This message was sent by Atlassian Jira (v8.20.7#820007)
[jira] [Commented] (ATLAS-4246) Make Kafka Interface aware of Kafka Schema Registry
[ https://issues.apache.org/jira/browse/ATLAS-4246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17425044#comment-17425044 ] Viktor Somogyi-Vass commented on ATLAS-4246: +1, reviewed it, looks good. > Make Kafka Interface aware of Kafka Schema Registry > --- > > Key: ATLAS-4246 > URL: https://issues.apache.org/jira/browse/ATLAS-4246 > Project: Atlas > Issue Type: Improvement > Components: kafka-integration >Affects Versions: 2.1.0, 3.0.0 >Reporter: Aileen Toleikis > Assignee: Viktor Somogyi-Vass >Priority: Major > Labels: Kafka > Fix For: 3.0.0, 2.3.0 > > > Kafka Community is using Schema Registry more and more heavily but as Atlas > is currently unaware of this, this extension helps Atlas make use of the > Schemas. > > We have tested this extension and we have production environments where Atlas > will not be allowed without schema registry access. We have received feedback > that this extension would be sufficient to allow production use. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (ATLAS-4440) Upgrade Atlas's Kafka dependency to 2.8
[ https://issues.apache.org/jira/browse/ATLAS-4440?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17424342#comment-17424342 ] Viktor Somogyi-Vass commented on ATLAS-4440: [~sarath] yes, it is fully backward compatible, there are no issues between client-server communication. Although new protocols were added Kafka is able to handle it via its protocol handshake mechanism so there will be no new issues when older clients connect to newer brokers or vice-versa. This protocol compatibility still goes back to 0.10 for both clients and brokers. Also in this version the "zookeeperless" mode was introduced but for now that is only a beta feature and it won't be enabled by default. > Upgrade Atlas's Kafka dependency to 2.8 > --- > > Key: ATLAS-4440 > URL: https://issues.apache.org/jira/browse/ATLAS-4440 > Project: Atlas > Issue Type: Improvement > Components: atlas-core >Reporter: Viktor Somogyi-Vass > Assignee: Viktor Somogyi-Vass >Priority: Major > Attachments: 0001-ATLAS-4440-Update-Kafka-dependency-to-2.8.1.patch > > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Created] (ATLAS-4440) Upgrade Atlas's Kafka dependency to 2.8
Viktor Somogyi-Vass created ATLAS-4440: -- Summary: Upgrade Atlas's Kafka dependency to 2.8 Key: ATLAS-4440 URL: https://issues.apache.org/jira/browse/ATLAS-4440 Project: Atlas Issue Type: Improvement Components: atlas-core Reporter: Viktor Somogyi-Vass Assignee: Viktor Somogyi-Vass -- This message was sent by Atlassian Jira (v8.3.4#803005)
Re: Review Request 73286: Initial commit
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73286/#review222880 --- Reviewed your code and made some smaller comments. I think overall this is a very useful feature to add. - Viktor Somogyi-Vass On April 27, 2021, 10:28 a.m., Aileen Toleikis wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73286/ > --- > > (Updated April 27, 2021, 10:28 a.m.) > > > Review request for atlas. > > > Repository: atlas > > > Description > --- > > The Kafka Community is using Schema Registry more and more heavily but as > Atlas is currently unaware of this, this extension helps Atlas make use of > the Schemas. > > We have tested this extension and we have production environments where Atlas > will not be allowed without schema registry access. We have received feedback > that this extension would be sufficient to allow production use. > > Changes: > > initial commit > > > added schema creation in Atlas > > > added Schema creation, Schema linkage and started to work on versions > > > avro_field creation is working. However, there is no connection to schemas yet > > > smaller refactorings and improved logging > > > added field reference in schemas > > > deleted second README and added gitignore > > > Smaller changes after rebasing on new version > > > moved HTTP REST calls to a separate class (SchemaRegistryConnector.java) > > > added testCreateSchema() as test case > > > added testCreateField() and testUpdateField() as test case > > > added testGetSchemas() and testGetSchemaVersions() as test case > > > added testUpdateSchema() as test case > > > removed unnecessary comments > > > Diffs > - > > addons/kafka-bridge/pom.xml 7fe97eb39897e32f5e2a4104cc06948e1ec48b27 > > addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java > bf74c67454120e3bdeebab0d7cc428d6bba8c021 > > addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/SchemaRegistryConnector.java > PRE-CREATION > > addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/model/KafkaDataTypes.java > 0f81b4c37a1d8f84c587a914eb460b290ae36987 > > addons/kafka-bridge/src/test/java/org/apache/atlas/kafka/bridge/KafkaBridgeTest.java > f86ceb58fd250d80f3ae3c71085c8c0adbc9116b > > > Diff: https://reviews.apache.org/r/73286/diff/1/ > > > Testing > --- > > > Thanks, > > Aileen Toleikis > >
Re: Review Request 73286: Initial commit
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73286/#review222879 --- addons/kafka-bridge/pom.xml Lines 143 (patched) <https://reviews.apache.org/r/73286/#comment312051> Please introduce a property in the root pom and use the reference here. addons/kafka-bridge/pom.xml Lines 148 (patched) <https://reviews.apache.org/r/73286/#comment312050> Could you please use the ${httpcomponents-httpclient.version} property here from the root pom? Also as I see the DefaultHttpClient has been deprecated since, so it might require some rework in the code too. addons/kafka-bridge/pom.xml Lines 153 (patched) <https://reviews.apache.org/r/73286/#comment312052> Please use the ${hive.version} property here. addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java Lines 67 (patched) <https://reviews.apache.org/r/73286/#comment312053> Please group public and private fields together. addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java Lines 503 (patched) <https://reviews.apache.org/r/73286/#comment312056> Can we reuse this HttpClient instead of creating a new one each time? addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/SchemaRegistryConnector.java Lines 38 (patched) <https://reviews.apache.org/r/73286/#comment312059> What happens if there are no schema registry running? Does it throw a connection exception or does handle this? addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/SchemaRegistryConnector.java Lines 51 (patched) <https://reviews.apache.org/r/73286/#comment312057> `httpClient` is passed in via an argument which would assume that this is a resource handled by the caller. I think it would be nicer to close this instead in the caller in a `finally` block. addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/SchemaRegistryConnector.java Lines 62 (patched) <https://reviews.apache.org/r/73286/#comment312058> Which methods throw IOException and UnsupportedOperationException? Do we have to handle them here? - Viktor Somogyi-Vass On April 27, 2021, 10:28 a.m., Aileen Toleikis wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73286/ > --- > > (Updated April 27, 2021, 10:28 a.m.) > > > Review request for atlas. > > > Repository: atlas > > > Description > --- > > The Kafka Community is using Schema Registry more and more heavily but as > Atlas is currently unaware of this, this extension helps Atlas make use of > the Schemas. > > We have tested this extension and we have production environments where Atlas > will not be allowed without schema registry access. We have received feedback > that this extension would be sufficient to allow production use. > > Changes: > > initial commit > > > added schema creation in Atlas > > > added Schema creation, Schema linkage and started to work on versions > > > avro_field creation is working. However, there is no connection to schemas yet > > > smaller refactorings and improved logging > > > added field reference in schemas > > > deleted second README and added gitignore > > > Smaller changes after rebasing on new version > > > moved HTTP REST calls to a separate class (SchemaRegistryConnector.java) > > > added testCreateSchema() as test case > > > added testCreateField() and testUpdateField() as test case > > > added testGetSchemas() and testGetSchemaVersions() as test case > > > added testUpdateSchema() as test case > > > removed unnecessary comments > > > Diffs > - > > addons/kafka-bridge/pom.xml 7fe97eb39897e32f5e2a4104cc06948e1ec48b27 > > addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java > bf74c67454120e3bdeebab0d7cc428d6bba8c021 > > addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/SchemaRegistryConnector.java > PRE-CREATION > > addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/model/KafkaDataTypes.java > 0f81b4c37a1d8f84c587a914eb460b290ae36987 > > addons/kafka-bridge/src/test/java/org/apache/atlas/kafka/bridge/KafkaBridgeTest.java > f86ceb58fd250d80f3ae3c71085c8c0adbc9116b > > > Diff: https://reviews.apache.org/r/73286/diff/1/ > > > Testing > --- > > > Thanks, > > Aileen Toleikis > >
[jira] [Assigned] (ATLAS-3929) Kafka hook for Atlas to capture production/consumption
[ https://issues.apache.org/jira/browse/ATLAS-3929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass reassigned ATLAS-3929: -- Assignee: Viktor Somogyi-Vass > Kafka hook for Atlas to capture production/consumption > -- > > Key: ATLAS-3929 > URL: https://issues.apache.org/jira/browse/ATLAS-3929 > Project: Atlas > Issue Type: Improvement > Components: kafka-integration >Reporter: Dennis Jaheruddin > Assignee: Viktor Somogyi-Vass >Priority: Major > > Currently we already have hooks for many engines (such as Spark, Nifi, > Flink). We also have a hook into the classical storage (HDFS). However, with > streaming becoming more and more dominant, we should also get a hook into the > streaming storage: Kafka. > Currently producers and consumers need to explicitly take atlas into account, > but for stronger governance and auditability, it should be possible to > capture every interaction with Kafka, regardless of how consumption or > production is done. -- This message was sent by Atlassian Jira (v8.3.4#803005)
Re: Review Request 72729: ATLAS-3910: Use KafkaZkClient instead of ZkUtils
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72729/#review221686 --- Ship it! Ship It! - Viktor Somogyi-Vass On Aug. 11, 2020, 7:42 a.m., Viktor Somogyi-Vass wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72729/ > --- > > (Updated Aug. 11, 2020, 7:42 a.m.) > > > Review request for atlas and Madhan Neethiraj. > > > Repository: atlas > > > Description > --- > > ATLAS-3910: Use KafkaZkClient instead of ZkUtils > > > Diffs > - > > > addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java > 40b1fee71e57175cd7ddf8a3696ef147282295c2 > > addons/kafka-bridge/src/test/java/org/apache/atlas/kafka/bridge/KafkaBridgeTest.java > c8cc85cda1f62b2d1a0a49c61abf06d9beb6fdb7 > notification/src/main/java/org/apache/atlas/hook/AtlasTopicCreator.java > c69574103e360fcfb7dfb525bb06043a1fd647c3 > notification/src/test/java/org/apache/atlas/hook/AtlasTopicCreatorTest.java > 293784770502299875177bd4a4a9c4b32c6b5a9a > > > Diff: https://reviews.apache.org/r/72729/diff/2/ > > > Testing > --- > > > Thanks, > > Viktor Somogyi-Vass > >
Re: Review Request 72729: ATLAS-3910: Use KafkaZkClient instead of ZkUtils
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72729/ --- (Updated Aug. 11, 2020, 7:42 a.m.) Review request for atlas and Madhan Neethiraj. Repository: atlas Description --- ATLAS-3910: Use KafkaZkClient instead of ZkUtils Diffs (updated) - addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java 40b1fee71e57175cd7ddf8a3696ef147282295c2 addons/kafka-bridge/src/test/java/org/apache/atlas/kafka/bridge/KafkaBridgeTest.java c8cc85cda1f62b2d1a0a49c61abf06d9beb6fdb7 notification/src/main/java/org/apache/atlas/hook/AtlasTopicCreator.java c69574103e360fcfb7dfb525bb06043a1fd647c3 notification/src/test/java/org/apache/atlas/hook/AtlasTopicCreatorTest.java 293784770502299875177bd4a4a9c4b32c6b5a9a Diff: https://reviews.apache.org/r/72729/diff/2/ Changes: https://reviews.apache.org/r/72729/diff/1-2/ Testing --- Thanks, Viktor Somogyi-Vass
Re: Asking for JIRA permissions
Thanks Ramesh! On Thu, Aug 6, 2020 at 4:03 PM Ramesh Mani wrote: > Vitor, > > I am reviewing this change and will get back soon. > > Thanks, > Ramesh > > On Thu, Aug 6, 2020 at 6:50 AM Viktor Somogyi-Vass > wrote: > > > Who would be the target audience? I've added the atlas group but do I > need > > to add specific people? > > This is the review: https://reviews.apache.org/r/72729/ > > > > Thanks, > > Viktor > > > > On Mon, Aug 3, 2020 at 4:27 PM Viktor Somogyi-Vass < > > viktor.somo...@cloudera.com> wrote: > > > > > Hi Madhan, > > > > > > Thanks, I'll submit my patch through Reviewboard then. > > > > > > Viktor > > > > > > On Tue, Jul 28, 2020 at 9:00 AM Madhan Neethiraj > > > wrote: > > > > > >> Viktor, > > >> > > >> Thanks for your interest in contributing to Apache Atlas. I added you > as > > >> a contributor to Apache Atlas. You can now assign JIRAs to yourself, > and > > >> add patches. > > >> > > >> Review board (https://reviews.apache.org/) is preferred by most Atlas > > >> folks to review and provide feedback on the patches. > > >> > > >> Regards, > > >> Madhan > > >> > > >> > > >> > > >> > > >> On 6/25/20, 2:56 AM, "Viktor Somogyi-Vass" < > > viktor.somo...@cloudera.com.INVALID> > > >> wrote: > > >> > > >> Hi devs, > > >> > > >> I created a JIRA [1] that I'd like to resolve (and have a PR > > already) > > >> but > > >> I'd like to assign it to myself. Could you please grant me the > > >> required permissions? > > >> Also, do you accept PR's against the master branch or does it have > > to > > >> be > > >> provided via JIRA in git patch format? > > >> > > >> [1] > > >> https://issues.apache.org/jira/browse/ATLAS-3864 > > >> > > >> Thanks, > > >> Viktor > > >> > > >> > > >> > > >
Re: Asking for JIRA permissions
Who would be the target audience? I've added the atlas group but do I need to add specific people? This is the review: https://reviews.apache.org/r/72729/ Thanks, Viktor On Mon, Aug 3, 2020 at 4:27 PM Viktor Somogyi-Vass < viktor.somo...@cloudera.com> wrote: > Hi Madhan, > > Thanks, I'll submit my patch through Reviewboard then. > > Viktor > > On Tue, Jul 28, 2020 at 9:00 AM Madhan Neethiraj > wrote: > >> Viktor, >> >> Thanks for your interest in contributing to Apache Atlas. I added you as >> a contributor to Apache Atlas. You can now assign JIRAs to yourself, and >> add patches. >> >> Review board (https://reviews.apache.org/) is preferred by most Atlas >> folks to review and provide feedback on the patches. >> >> Regards, >> Madhan >> >> >> >> >> On 6/25/20, 2:56 AM, "Viktor Somogyi-Vass" >> >> wrote: >> >> Hi devs, >> >> I created a JIRA [1] that I'd like to resolve (and have a PR already) >> but >> I'd like to assign it to myself. Could you please grant me the >> required permissions? >> Also, do you accept PR's against the master branch or does it have to >> be >> provided via JIRA in git patch format? >> >> [1] >> https://issues.apache.org/jira/browse/ATLAS-3864 >> >> Thanks, >> Viktor >> >> >>
Review Request 72729: ATLAS-3910: Use KafkaZkClient instead of ZkUtils
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72729/ --- Review request for atlas. Repository: atlas Description --- ATLAS-3910: Use KafkaZkClient instead of ZkUtils Diffs - addons/kafka-bridge/src/main/java/org/apache/atlas/kafka/bridge/KafkaBridge.java 40b1fee71e57175cd7ddf8a3696ef147282295c2 addons/kafka-bridge/src/test/java/org/apache/atlas/kafka/bridge/KafkaBridgeTest.java c8cc85cda1f62b2d1a0a49c61abf06d9beb6fdb7 notification/src/main/java/org/apache/atlas/hook/AtlasTopicCreator.java c69574103e360fcfb7dfb525bb06043a1fd647c3 notification/src/test/java/org/apache/atlas/hook/AtlasTopicCreatorTest.java 293784770502299875177bd4a4a9c4b32c6b5a9a Diff: https://reviews.apache.org/r/72729/diff/1/ Testing --- Thanks, Viktor Somogyi-Vass
Re: Asking for JIRA permissions
Hi Madhan, Thanks, I'll submit my patch through Reviewboard then. Viktor On Tue, Jul 28, 2020 at 9:00 AM Madhan Neethiraj wrote: > Viktor, > > Thanks for your interest in contributing to Apache Atlas. I added you as a > contributor to Apache Atlas. You can now assign JIRAs to yourself, and add > patches. > > Review board (https://reviews.apache.org/) is preferred by most Atlas > folks to review and provide feedback on the patches. > > Regards, > Madhan > > > > > On 6/25/20, 2:56 AM, "Viktor Somogyi-Vass" > > wrote: > > Hi devs, > > I created a JIRA [1] that I'd like to resolve (and have a PR already) > but > I'd like to assign it to myself. Could you please grant me the > required permissions? > Also, do you accept PR's against the master branch or does it have to > be > provided via JIRA in git patch format? > > [1] > https://issues.apache.org/jira/browse/ATLAS-3864 > > Thanks, > Viktor > > >
[jira] [Updated] (ATLAS-3864) Atlas should use AdminClient where possible
[ https://issues.apache.org/jira/browse/ATLAS-3864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-3864: --- Description: Zookeeper access should be restricted wherever possible as going forward Kafka won't use Zookeeper (see [KIP-500|https://cwiki.apache.org/confluence/display/KAFKA/KIP-500%3A+Replace+ZooKeeper+with+a+Self-Managed+Metadata+Quorum]). Also the Zookeeper classes in Kafka aren't public apis so usage should be avoided wherever possible. As a tactical fix (see ATLAS-3910) though it is possible to continue using KafkaZkClient as that class likely won't change much until the completion of KIP-500. was: Zookeeper access should be restricted wherever possible as going forward Kafka won't use Zookeeper (see [KIP-500|https://cwiki.apache.org/confluence/display/KAFKA/KIP-500%3A+Replace+ZooKeeper+with+a+Self-Managed+Metadata+Quorum]). Also the Zookeeper classes in Kafka aren't public apis so usage should be avoided wherever possible. As a tactical fix though it is possible to continue > Atlas should use AdminClient where possible > --- > > Key: ATLAS-3864 > URL: https://issues.apache.org/jira/browse/ATLAS-3864 > Project: Atlas > Issue Type: Improvement > Reporter: Viktor Somogyi-Vass >Priority: Major > > Zookeeper access should be restricted wherever possible as going forward > Kafka won't use Zookeeper (see > [KIP-500|https://cwiki.apache.org/confluence/display/KAFKA/KIP-500%3A+Replace+ZooKeeper+with+a+Self-Managed+Metadata+Quorum]). > Also the Zookeeper classes in Kafka aren't public apis so usage should be > avoided wherever possible. As a tactical fix (see ATLAS-3910) though it is > possible to continue using KafkaZkClient as that class likely won't change > much until the completion of KIP-500. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ATLAS-3864) Atlas should use AdminClient where possible
[ https://issues.apache.org/jira/browse/ATLAS-3864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-3864: --- Description: Zookeeper access should be restricted wherever possible as going forward Kafka won't use Zookeeper (see [KIP-500|https://cwiki.apache.org/confluence/display/KAFKA/KIP-500%3A+Replace+ZooKeeper+with+a+Self-Managed+Metadata+Quorum]). Also the Zookeeper classes in Kafka aren't public apis so usage should be avoided wherever possible. As a tactical fix though it is possible to continue was:Zookeeper access should be restricted wherever possible as going forward Kafka won't use Zookeeper (see KIP-500). > Atlas should use AdminClient where possible > --- > > Key: ATLAS-3864 > URL: https://issues.apache.org/jira/browse/ATLAS-3864 > Project: Atlas > Issue Type: Improvement > Reporter: Viktor Somogyi-Vass >Priority: Major > > Zookeeper access should be restricted wherever possible as going forward > Kafka won't use Zookeeper (see > [KIP-500|https://cwiki.apache.org/confluence/display/KAFKA/KIP-500%3A+Replace+ZooKeeper+with+a+Self-Managed+Metadata+Quorum]). > Also the Zookeeper classes in Kafka aren't public apis so usage should be > avoided wherever possible. As a tactical fix though it is possible to > continue -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ATLAS-3864) Atlas should use AdminClient where possible
[ https://issues.apache.org/jira/browse/ATLAS-3864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-3864: --- Description: Zookeeper access should be restricted wherever possible as going forward Kafka won't use Zookeeper (see KIP-500). (was: Kafka removed ZkUtils as it's an internal class and replaced it with KafkaZkClient. To be compatible with newer Kafka versions Atlas should use AdminClient and migrate from ZkUtils to KafkaZkClient.) > Atlas should use AdminClient where possible > --- > > Key: ATLAS-3864 > URL: https://issues.apache.org/jira/browse/ATLAS-3864 > Project: Atlas > Issue Type: Improvement > Reporter: Viktor Somogyi-Vass >Priority: Major > > Zookeeper access should be restricted wherever possible as going forward > Kafka won't use Zookeeper (see KIP-500). -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Created] (ATLAS-3910) Use KafkaZkClient instead of ZkUtils
Viktor Somogyi-Vass created ATLAS-3910: -- Summary: Use KafkaZkClient instead of ZkUtils Key: ATLAS-3910 URL: https://issues.apache.org/jira/browse/ATLAS-3910 Project: Atlas Issue Type: Improvement Reporter: Viktor Somogyi-Vass Assignee: Viktor Somogyi-Vass Kafka removed ZkUtils from 2.5 as it's an internal class and replaced it with KafkaZkClient. To be compatible with newer Kafka versions Atlas shouldn't use ZkUtils. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ATLAS-3864) Atlas should use AdminClient where possible
[ https://issues.apache.org/jira/browse/ATLAS-3864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-3864: --- Summary: Atlas should use AdminClient where possible (was: Atlas should use AdminClient where possible and remove ZkUtils) > Atlas should use AdminClient where possible > --- > > Key: ATLAS-3864 > URL: https://issues.apache.org/jira/browse/ATLAS-3864 > Project: Atlas > Issue Type: Improvement > Reporter: Viktor Somogyi-Vass >Priority: Major > > Kafka removed ZkUtils as it's an internal class and replaced it with > KafkaZkClient. To be compatible with newer Kafka versions Atlas should use > AdminClient and migrate from ZkUtils to KafkaZkClient. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ATLAS-3864) Atlas should use AdminClient where possible and remove ZkUtils
[ https://issues.apache.org/jira/browse/ATLAS-3864?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-3864: --- Summary: Atlas should use AdminClient where possible and remove ZkUtils (was: KafkaBridge should use AdminClient) > Atlas should use AdminClient where possible and remove ZkUtils > -- > > Key: ATLAS-3864 > URL: https://issues.apache.org/jira/browse/ATLAS-3864 > Project: Atlas > Issue Type: Improvement > Reporter: Viktor Somogyi-Vass >Priority: Major > > Kafka removed ZkUtils as it's an internal class and replaced it with > KafkaZkClient. To be compatible with newer Kafka versions Atlas should use > AdminClient and migrate from ZkUtils to KafkaZkClient. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ATLAS-3865) HookConsumer shouldn't use ShutdownThrea
[ https://issues.apache.org/jira/browse/ATLAS-3865?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-3865: --- Summary: HookConsumer shouldn't use ShutdownThrea (was: HookConsumer shouldn't use ShutdownThread or depend on Scala code) > HookConsumer shouldn't use ShutdownThrea > > > Key: ATLAS-3865 > URL: https://issues.apache.org/jira/browse/ATLAS-3865 > Project: Atlas > Issue Type: Improvement > Reporter: Viktor Somogyi-Vass >Priority: Major > > The internal ShutdownThread depends on Kafka core which isn't desired as it > isn't a public API and also it pulls in a Scala dependency that can change > across Kafka versions and isn't good to expose. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ATLAS-3865) HookConsumer shouldn't use ShutdownThread
[ https://issues.apache.org/jira/browse/ATLAS-3865?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-3865: --- Summary: HookConsumer shouldn't use ShutdownThread (was: HookConsumer shouldn't use ShutdownThrea) > HookConsumer shouldn't use ShutdownThread > - > > Key: ATLAS-3865 > URL: https://issues.apache.org/jira/browse/ATLAS-3865 > Project: Atlas > Issue Type: Improvement > Reporter: Viktor Somogyi-Vass >Priority: Major > > The internal ShutdownThread depends on Kafka core which isn't desired as it > isn't a public API and also it pulls in a Scala dependency that can change > across Kafka versions and isn't good to expose. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (ATLAS-3865) HookConsumer shouldn't use ShutdownThread or depend on Scala code
[ https://issues.apache.org/jira/browse/ATLAS-3865?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Viktor Somogyi-Vass updated ATLAS-3865: --- Summary: HookConsumer shouldn't use ShutdownThread or depend on Scala code (was: HookConsumer shouldn't use ShutdownThread) > HookConsumer shouldn't use ShutdownThread or depend on Scala code > - > > Key: ATLAS-3865 > URL: https://issues.apache.org/jira/browse/ATLAS-3865 > Project: Atlas > Issue Type: Improvement > Reporter: Viktor Somogyi-Vass >Priority: Major > > The internal ShutdownThread depends on Kafka core which isn't desired as it > isn't a public API and also it pulls in a Scala dependency that can change > across Kafka versions and isn't good to expose. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Created] (ATLAS-3865) HookConsumer shouldn't use ShutdownThread
Viktor Somogyi-Vass created ATLAS-3865: -- Summary: HookConsumer shouldn't use ShutdownThread Key: ATLAS-3865 URL: https://issues.apache.org/jira/browse/ATLAS-3865 Project: Atlas Issue Type: Improvement Reporter: Viktor Somogyi-Vass The internal ShutdownThread depends on Kafka core which isn't desired as it isn't a public API and also it pulls in a Scala dependency that can change across Kafka versions and isn't good to expose. -- This message was sent by Atlassian Jira (v8.3.4#803005)
Asking for JIRA permissions
Hi devs, I created a JIRA [1] that I'd like to resolve (and have a PR already) but I'd like to assign it to myself. Could you please grant me the required permissions? Also, do you accept PR's against the master branch or does it have to be provided via JIRA in git patch format? [1] https://issues.apache.org/jira/browse/ATLAS-3864 Thanks, Viktor
[jira] [Created] (ATLAS-3864) KafkaBridge should use AdminClient
Viktor Somogyi-Vass created ATLAS-3864: -- Summary: KafkaBridge should use AdminClient Key: ATLAS-3864 URL: https://issues.apache.org/jira/browse/ATLAS-3864 Project: Atlas Issue Type: Improvement Reporter: Viktor Somogyi-Vass Kafka removed ZkUtils as it's an internal class and replaced it with KafkaZkClient. To be compatible with newer Kafka versions Atlas should use AdminClient and migrate from ZkUtils to KafkaZkClient. -- This message was sent by Atlassian Jira (v8.3.4#803005)