Build failed in Jenkins: Atlas-master-IntegrationTests #200

2017-11-30 Thread Apache Jenkins Server
See Changes: [madhan] ATLAS-2258: Fix new Integration test failures introduced due to [madhan] ATLAS-2290: fix for unit test failure in HiveMetaStoreBridgeTest

Jenkins build is back to stable : Atlas-master-UnitTests #182

2017-11-30 Thread Apache Jenkins Server
See

[jira] [Commented] (ATLAS-2290) Unit test failure in HiveMetaStoreBridgeTest

2017-11-30 Thread Sarath Subramanian (JIRA)
[ https://issues.apache.org/jira/browse/ATLAS-2290?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16274046#comment-16274046 ] Sarath Subramanian commented on ATLAS-2290: --- +1 for the patch > Unit test failure in

[jira] [Updated] (ATLAS-2290) Unit test failure in HiveMetaStoreBridgeTest

2017-11-30 Thread Madhan Neethiraj (JIRA)
[ https://issues.apache.org/jira/browse/ATLAS-2290?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Madhan Neethiraj updated ATLAS-2290: Attachment: ATLAS-2290.patch > Unit test failure in HiveMetaStoreBridgeTest >

[jira] [Created] (ATLAS-2290) Unit test failure in HiveMetaStoreBridgeTest

2017-11-30 Thread Madhan Neethiraj (JIRA)
Madhan Neethiraj created ATLAS-2290: --- Summary: Unit test failure in HiveMetaStoreBridgeTest Key: ATLAS-2290 URL: https://issues.apache.org/jira/browse/ATLAS-2290 Project: Atlas Issue Type:

Build failed in Jenkins: Atlas-0.8-IntegrationTests #141

2017-11-30 Thread Apache Jenkins Server
See Changes: [madhan] ATLAS-2289: separate embedded kafka/zookeeper start/stop from -- [...truncated 527.47 KB...] 127.0.0.1 - - [01/Dec/2017:06:49:29 +]

Build failed in Jenkins: Atlas-master-IntegrationTests #199

2017-11-30 Thread Apache Jenkins Server
See Changes: [madhan] ATLAS-2289: separate embedded kafka/zookeeper start/stop from -- [...truncated 385.60 KB...] at

Jenkins build is still unstable: Atlas-master-UnitTests #181

2017-11-30 Thread Apache Jenkins Server
See

Re: Review Request 64235: ATLAS-2289: separate embedded kafka/zookeeper start/stop from KafkaNotification

2017-11-30 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64235/ --- (Updated Dec. 1, 2017, 5:02 a.m.) Review request for atlas. Changes ---

Re: Review Request 64235: ATLAS-2289: separate embedded kafka/zookeeper start/stop from KafkaNotification

2017-11-30 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64235/#review192427 --- Ship it! Ship It! - Sarath Subramanian On Nov. 30, 2017,

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Nigel Jones
On 2017-12-01 00:52, "Nigel Jones" wrote: > > atlas.server.ha.zookeeper.connect=localhost:2181 Reading the posts from earlier I then added 00:57 $ export MANAGE_LOCAL_SOLR=true 00:57 $ export MANAGE_LOCAL_HBASE=true 00:58 $ echo

[jira] [Updated] (ATLAS-2289) Embedded kafka/zookeeper server start/stop code to be moved out of KafkaNotification implementation

2017-11-30 Thread Madhan Neethiraj (JIRA)
[ https://issues.apache.org/jira/browse/ATLAS-2289?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Madhan Neethiraj updated ATLAS-2289: Attachment: ATLAS-2289.patch Review board URL: https://reviews.apache.org/r/64235/ >

Review Request 64235: ATLAS-2289: separate embedded kafka/zookeeper start/stop from KafkaNotification

2017-11-30 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64235/ --- Review request for atlas. Bugs: ATLAS-2289

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Nigel Jones
On 2017-12-01 00:31, Madhan Neethiraj wrote: > To get rid of “Connection refused” WARN logs, please set the following > property in conf/atlas-application.properties and start Atlas > atlas.server.ha.zookeeper.connect=localhost:2181 This line was commented out. I

[jira] [Created] (ATLAS-2289) Embedded kafka/zookeeper server start/stop code to be moved out of KafkaNotification implementation

2017-11-30 Thread Madhan Neethiraj (JIRA)
Madhan Neethiraj created ATLAS-2289: --- Summary: Embedded kafka/zookeeper server start/stop code to be moved out of KafkaNotification implementation Key: ATLAS-2289 URL:

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Madhan Neethiraj
Nigel, To get rid of “Connection refused” WARN logs, please set the following property in conf/atlas-application.properties and start Atlas : atlas.server.ha.zookeeper.connect=localhost:2181 Not sure what causes HBase error “ConnectionLoss for /hbase/meta-region-server” in your

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Nigel Jones
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64141/#review192410 --- I tried the new patch (2287.2) - mvn command as above (janus).

Build failed in Jenkins: Atlas-master-IntegrationTests #198

2017-11-30 Thread Apache Jenkins Server
See Changes: [ssubramanian] ATLAS-2287: Include lucene libraries when building atlas distribution -- [...truncated 384.97 KB...] at

Jenkins build is still unstable: Atlas-master-UnitTests #180

2017-11-30 Thread Apache Jenkins Server
See

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64141/#review192372 --- Ship it! Ship It! - Madhan Neethiraj On Nov. 30, 2017, 8:59

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64141/ --- (Updated Nov. 30, 2017, 12:59 p.m.) Review request for atlas, Apoorv Naik,

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Sarath Subramanian
> On Nov. 30, 2017, 6:47 a.m., David Radley wrote: > > pom.xml > > Line 673 (original), 674 (patched) > > > > > > The titan0 build failed for me because it was looking for JanusGraph. I > > think that this was

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Madhan Neethiraj
(Review board wouldn’t post my comments to this review – probably because of other updates to the review while I was reviewing. Hence sending my comments by email) >> David: The titan0 build failed for me because it was looking for JanusGraph. >> I think that this was because the configuration

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 5:06 p.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/ffdc/OCFErrorCode.java > > Lines 100 (patched) > > > > > > Shouldn't this br iterator rather than enumeration

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Sarath Subramanian
> On Nov. 30, 2017, 6:47 a.m., David Radley wrote: > > pom.xml > > Line 713 (original), 716 (patched) > > > > > > I could start the UI but had lots of Zookeeper exceptions and posts to > > create entities did not

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 5:06 p.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/Endpoint.java > > Lines 60 (patched) > > > > > > The field name encrypted implies that it is a boolean.

Build failed in Jenkins: Atlas-master-IntegrationTests #197

2017-11-30 Thread Apache Jenkins Server
See Changes: [madhan] ATLAS-2281: UI fix to address error in saving tag/type attribute filter -- [...truncated 385.62 KB...] at

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Madhan Neethiraj
> On Nov. 30, 2017, 2:47 p.m., David Radley wrote: > > pom.xml > > Line 713 (original), 716 (patched) > > > > > > I could start the UI but had lots of Zookeeper exceptions and posts to > > create entities did not

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 5:06 p.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/Meaning.java > > Lines 31 (patched) > > > > > > should be include the guid of the term here. That would

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 5:06 p.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/License.java > > Lines 251 (patched) > > > > > > I am wondering why the license conditions is plural

Jenkins build is unstable: Atlas-master-UnitTests #179

2017-11-30 Thread Apache Jenkins Server
See

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 5:27 p.m., Madhan Neethiraj wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/AdditionalProperties.java > > Lines 41 (patched) > > > > > > >> These properties are not used in

Build failed in Jenkins: Atlas-0.8-IntegrationTests #140

2017-11-30 Thread Apache Jenkins Server
See Changes: [madhan] ATLAS-2281: UI fix to address error in saving tag/type attribute filter -- [...truncated 529.33 KB...] 127.0.0.1 - - [30/Nov/2017:17:48:30

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Madhan Neethiraj
> On Nov. 30, 2017, 2:47 p.m., David Radley wrote: > > pom.xml > > Line 713 (original), 716 (patched) > > > > > > I could start the UI but had lots of Zookeeper exceptions and posts to > > create entities did not

Re: Review Request 64210: ATLAS-2281 : Saving Tag/Type attribute filter queries with null/not null filters.

2017-11-30 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64210/#review192311 --- Ship it! Ship It! - Madhan Neethiraj On Nov. 30, 2017, 1:50

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63503/#review192307 ---

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63503/#review192295 ---

compilation error in master

2017-11-30 Thread Peter Gergo Barna
Hi, I get compilation errors while building the master branch [ERROR] COMPILATION ERROR : [INFO] - [ERROR] /Users/pbarna/Desktop/apache-source/atlas/intg/src/main/java/org/apache/atlas/model/SearchFilter.java:[20,32] package

[jira] [Updated] (ATLAS-2249) Upgrade Atlas Client to use Jersey 2

2017-11-30 Thread Pierre Padovani (JIRA)
[ https://issues.apache.org/jira/browse/ATLAS-2249?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Pierre Padovani updated ATLAS-2249: --- Attachment: client-jersey2-V4.patch Updates to the patch based on current master changes. >

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Nigel Jones
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64141/#review192257 --- I tried this patch on atlas-master (0900 UTC 20171130), maven

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64141/#review192267 --- pom.xml Line 673 (original), 674 (patched)

[jira] [Created] (ATLAS-2288) NoClassDefFoundError Exception while running import-hive script when hbase table is create via Hive

2017-11-30 Thread Nixon Rodrigues (JIRA)
Nixon Rodrigues created ATLAS-2288: -- Summary: NoClassDefFoundError Exception while running import-hive script when hbase table is create via Hive Key: ATLAS-2288 URL:

[jira] [Updated] (ATLAS-2288) NoClassDefFoundError Exception while running import-hive script when hbase table is created via Hive

2017-11-30 Thread Nixon Rodrigues (JIRA)
[ https://issues.apache.org/jira/browse/ATLAS-2288?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nixon Rodrigues updated ATLAS-2288: --- Summary: NoClassDefFoundError Exception while running import-hive script when hbase table is

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 29, 2017, 4 p.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/AssetPropertyBase.java > > Lines 38 (patched) > > > > > > I am unsure what the thinking is here - the super

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 13, 2017, 9:07 a.m., Yao Li wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/Connection.java > > Lines 44 (patched) > > > > > > this part is not consistent with the ConnectorBroker code.

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 14, 2017, 12:25 p.m., Yao Li wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/Connection.java > > Lines 27 (patched) > > > > > > model 0101 does not exist in this > >

[jira] [Updated] (ATLAS-2281) Saving Tag/Type attribute filter queries with null/not null filters.

2017-11-30 Thread pratik pandey (JIRA)
[ https://issues.apache.org/jira/browse/ATLAS-2281?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] pratik pandey updated ATLAS-2281: - Fix Version/s: 0.8.2 1.0.0 > Saving Tag/Type attribute filter queries with

Review Request 64210: ATLAS-2281 : Saving Tag/Type attribute filter queries with null/not null filters.

2017-11-30 Thread pratik pandey
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64210/ --- Review request for atlas, Ashutosh Mestry, keval bhatt, Madhan Neethiraj, and

Re: slack

2017-11-30 Thread Mandy Chessell
I agree with David. Although slack does have advantages over email - particularly in terms of managing channels for different topics, I would be reluctant to add it as another comms mechanism if all of the Slack messages were not automatically sent to the dev list. All the best Mandy

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 14, 2017, 10:06 a.m., Yao Li wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/ConnectorBroker.java > > Lines 134 (patched) > > > > > > Do you mean in the ConnectorProvider class, it doesn't > >

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 10:29 a.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/CommentType.java > > Lines 36 (patched) > > > > > > would it not be better to say > > > >

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 11:08 a.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/Annotations.java > > Lines 39 (patched) > > > > > > I think you mean annotaiotn not enumeration here.

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 10:59 a.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/ConnectorProviderBase.java > > Lines 35 (patched) > > > > > > I suggest only having a constructor that supplies

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 10:59 a.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/Annotations.java > > Lines 33 (patched) > > > > > > I suggest that we could remove this count logic - I

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 11:08 a.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/properties/CommentType.java > > Lines 44 (patched) > > > > > > I think we can make this and other enum

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 30, 2017, 10:59 a.m., David Radley wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/ConnectorBroker.java > > Lines 141 (patched) > > > > > > Maybe I have missed it - I am wondering how a user needs

Re: slack

2017-11-30 Thread Jean-Baptiste Onofré
Big +1, I remember to have raise the same point in Beam ;) Regards JB On 11/30/2017 12:13 PM, David Radley wrote: Hi , Having all communication on the dev list means there is one place to look to see how a community is communicating and is the way Apache project are supposed to work. If we

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 29, 2017, 4 p.m., David Radley wrote: > > om-fwk-ocf/src/test/java/org/apache/atlas/ocf/test/QuickTest.java > > Lines 45 (patched) > > > > > > I suggest moving these tests into junits so they can be run as

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Mandy Chessell
> On Nov. 14, 2017, 10:06 a.m., Yao Li wrote: > > om-fwk-ocf/src/main/java/org/apache/atlas/ocf/ConnectorBroker.java > > Lines 164 (patched) > > > > > > I didn't find this class's definiation in the patch

slack

2017-11-30 Thread David Radley
Hi , Having all communication on the dev list means there is one place to look to see how a community is communicating and is the way Apache project are supposed to work. If we start to use slack then we would need to to remember to copy the dev list for any non-trivial communications to ensure

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63503/#review192260 ---

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread David Radley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63503/#review192254 ---

Re: standalone mode setup

2017-11-30 Thread Nigel Jones
On 2017-11-30 09:40, Graham Wallis wrote: > It would be raising a JIRA for the documentation errors you spotted and Andrew - Some of the core documentation is currently in the build (ie the top level README.me, docs/*, and some module level documentation,

Re: Instructions to build & run Atlas in dev environment

2017-11-30 Thread Nigel Jones
On 2017-11-29 18:25, Madhan Neethiraj wrote: > Graham, David, > > Can you please review Sarath’s patch to resolve this issue > https://reviews.apache.org/r/64141/, and give your feedback? I also tried Sarath's patch on top of today's master, with only the mvn commands

Re: Review Request 63462: ATLAS-2245: Update maven dependency to 3.5.0

2017-11-30 Thread Nigel Jones
On 2017-11-30 09:56, Nigel Jones <jon...@uk.ibm.com> wrote: > I tried this patch on atlas-master (0900 UTC 20171130), maven 3.5.0, java 8 > (b152), macOS. Odd - that comment was in relation to a different patch. too many windows probably Please ignore

Re: Review Request 63462: ATLAS-2245: Update maven dependency to 3.5.0

2017-11-30 Thread Nigel Jones
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63462/#review192255 --- I tried this patch on atlas-master (0900 UTC 20171130), maven

Re: standalone mode setup

2017-11-30 Thread Graham Wallis
Hi Andrew Your git page looks very similar to the parallel discussion and will use JanusGraph by default - which for me works and starts up. Other people have got it working with Titan 0.5.4 but some have found that clearing out their local maven repo was necessary. I will probably try Titan

Re: Review Request 63503: ATLAS-1095 - Review of OCF - the Open Connector Framework

2017-11-30 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63503/#review192180 --- om-fwk-ocf/src/main/java/org/apache/atlas/ocf/ConnectorBase.java