://kafka.apache.org/0110/javadoc/org/apache/kafka/clients/consumer/KafkaConsumer.html
- Ashutosh Mestry
On Aug. 2, 2017, 8:48 a.m., Nixon Rodrigues wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61263/#review182038
---
Ship it!
Ship It!
- Ashutosh Mestry
On July 31, 2017, 7:07
the scenarios and verify the fix.
**Functional tests**
Verified regular notification scenarios.
Thanks,
Ashutosh Mestry
8/09/440a989c-81ea-4653-b896-9d0d51465de4__ugly.json
Pretty
https://reviews.apache.org/media/uploaded/files/2017/08/09/1cf92e66-2dc6-4ded-a546-f1eaa7dcba36__pretty.json
Thanks,
Ashutosh Mestry
for _NotificationHookConsumerTest_.
Thanks,
Ashutosh Mestry
tests**
- Executed existing unit tests.
**Functional tests**
- Executed standard set of tests.
Thanks,
Ashutosh Mestry
/
Changes: https://reviews.apache.org/r/60722/diff/1-2/
Testing
---
**Unit tests**
- Executed existing unit tests.
**Functional tests**
- Executed standard set of tests.
Thanks,
Ashutosh Mestry
)
<https://reviews.apache.org/r/60597/#comment255549>
Should this be 1000 ms? My thinking is 100 ms will be too short.
- Ashutosh Mestry
On July 3, 2017, 1:17 p.m., Nixon Rodrigues wrote:
>
> ---
> This is an automatica
Diff: https://reviews.apache.org/r/60857/diff/1/
Testing
---
**Unit tests**
Unit test verifying this behavior were added.
**Functional tests**
Regular imports were run.
Thanks,
Ashutosh Mestry
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60868/#review180535
---
Ship it!
Ship It!
- Ashutosh Mestry
On July 14, 2017, 3:13
ce.java
92217178
Diff: https://reviews.apache.org/r/61002/diff/2/
Changes: https://reviews.apache.org/r/61002/diff/1-2/
Testing
---
**Functional test**
Imported ZIP file using the new API.
**Unit tests**
None.
Thanks,
Ashutosh Mestry
ttps://reviews.apache.org/r/61046/diff/1/
Testing
---
**Unit tests**
- Updated unit tests.
- Executed existing tests.
**Function tests**
On local VM by changing application property and using debugger to step thru.
Thanks,
Ashutosh Mestry
it tests.
- Executed existing tests.
**Function tests**
On local VM by changing application property and using debugger to step thru.
Thanks,
Ashutosh Mestry
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61103/#review181353
---
Ship it!
Ship It!
- Ashutosh Mestry
On July 25, 2017, 11:30
/diff/3/
Changes: https://reviews.apache.org/r/60857/diff/2-3/
Testing
---
**Unit tests**
Unit test verifying this behavior were added.
**Functional tests**
Regular imports were run.
Thanks,
Ashutosh Mestry
ccedeb14
Diff: https://reviews.apache.org/r/60857/diff/2/
Changes: https://reviews.apache.org/r/60857/diff/1-2/
Testing
---
**Unit tests**
Unit test verifying this behavior were added.
**Functional tests**
Regular imports were run.
Thanks,
Ashutosh Mestry
/60816/diff/1/
Testing
---
**Unit tests**
Executed exisiting tests.
**Functional tests**
- Created entities via Hive.
Thanks,
Ashutosh Mestry
://reviews.apache.org/r/61518/diff/1/
Testing
---
**Unit tests**
Existing suite.
Thanks,
Ashutosh Mestry
/notification/NotificationHookConsumer.java
51276d3
Diff: https://reviews.apache.org/r/61195/diff/1/
Testing
---
**Unit tests**
None.
**Functional tests**
Verified the performance improvement using the _AtlasPerfTracer_.
Thanks,
Ashutosh Mestry
-log4j.xml c30403e
Diff: https://reviews.apache.org/r/61200/diff/1/
Testing
---
**Functional tests**
Observed configuration changes in the logs generated.
Thanks,
Ashutosh Mestry
pache.org/r/61665/#review183030
---
On Aug. 15, 2017, 4:58 p.m., Ashutosh Mestry wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revie
onal unit tests: _UniqueList_.
- Import-Export APIs verified with test file set.
Thanks,
Ashutosh Mestry
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61940/#review185305
---
Ship it!
Ship It!
- Ashutosh Mestry
On Sept. 13, 2017, 2:11
ATION
webapp/src/main/java/org/apache/atlas/notification/NotificationEntityChangeListener.java
6eadc967
webapp/src/main/java/org/apache/atlas/web/rest/DiscoveryREST.java 52258e3c
webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java c32f36ea
webapp/src/main/java/org/apache/atlas/web/util/Servlets.java 4a92763a
Diff: https://reviews.apache.org/r/62039/diff/5/
Changes: https://reviews.apache.org/r/62039/diff/4-5/
Testing
---
**Deployment**
- Copy _0010-base_model.json_ & _0011-user-profile_model.json_ to server
location: _/usr/hdp/current/atlas-server/models_
- Deploy _atlas.war_
**Unit tests**
Added unit tests for:
- _UserProfileService_.
**Functional testing**
- Verification using CURL calls.
- Verification via UI to ensure that internal entities do not show up.
Thanks,
Ashutosh Mestry
ws.apache.org/r/62382/#comment261877>
For my education: What is the implication of this change?
- Ashutosh Mestry
On Sept. 18, 2017, 1:06 p.m., Nixon Rodrigues wrote:
>
> ---
> This is an automatically generated e-mail. To reply, vi
ntityChangeListener.java
6eadc967
webapp/src/main/java/org/apache/atlas/web/rest/DiscoveryREST.java 52258e3c
webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java c32f36ea
webapp/src/main/java/org/apache/atlas/web/util/Servlets.java 4a92763a
Diff: https://reviews.apache.org/r/62039/diff/7/
Changes: https://reviews.apache.org/r/62039/diff/6-7/
Testing
---
**Deployment**
- Copy _0010-base_model.json_ & _0011-user-profile_model.json_ to server
location: _/usr/hdp/current/atlas-server/models_
- Deploy _atlas.war_
**Unit tests**
Added unit tests for:
- _UserProfileService_.
**Functional testing**
- Verification using CURL calls.
- Verification via UI to ensure that internal entities do not show up.
File Attachments
Patch for Master
https://reviews.apache.org/media/uploaded/files/2017/09/15/ba9b07c2-5868-4f79-ad51-7d7c9ee39c44__ATLAS-2100-Master-DataAccess.patch
Thanks,
Ashutosh Mestry
---
**Unit tests**
- Executed impacted unit tests.
- Build with tests.
Thanks,
Ashutosh Mestry
d-28f6-487a-8126-aaf6fc8a1028__savedSearch.json
saved search for update
https://reviews.apache.org/media/uploaded/files/2017/09/19/2b1d4619-65d8-4435-87cb-9b11878a7420__savedSearch-for-update.json
Thanks,
Ashutosh Mestry
s/2017/09/19/f9f6ec2d-28f6-487a-8126-aaf6fc8a1028__savedSearch.json
saved search for update
https://reviews.apache.org/media/uploaded/files/2017/09/19/2b1d4619-65d8-4435-87cb-9b11878a7420__savedSearch-for-update.json
Thanks,
Ashutosh Mestry
rom the new UI.
CURL calls.
File Attachments
----
Saved Search JSON
https://reviews.apache.org/media/uploaded/files/2017/09/19/f845e2cf-3354-48e4-bfed-76b09a581af2__savedSearch.json
Thanks,
Ashutosh Mestry
views.apache.org/r/62420/diff/2/
Changes: https://reviews.apache.org/r/62420/diff/1-2/
Testing
---
**Unit tests**
Updated to reflect new logic.
**Functional tests**
>From the new UI.
CURL calls.
File Attachments
----
Saved Search JSON
https://reviews.apache.org/media/uploaded/files/2017/09/19/f845e2cf-3354-48e4-bfed-76b09a581af2__savedSearch.json
Thanks,
Ashutosh Mestry
saved/execute/18a21a1b-af61-47c6-a50a-34e8ae81dad3;
```
Diffs
-
webapp/src/main/java/org/apache/atlas/web/rest/DiscoveryREST.java aed82ae0
Diff: https://reviews.apache.org/r/62484/diff/1/
Testing
---
**Unit tests**
None
**Functional tests**
Executed APIs using CURL calls.
Thanks,
Ashutosh Mestry
aed82ae0
Diff: https://reviews.apache.org/r/62484/diff/1/
Testing (updated)
---
**Unit tests**
Executed existing tests.
**Functional tests**
Executed APIs using CURL calls.
Thanks,
Ashutosh Mestry
84/diff/2/
Changes: https://reviews.apache.org/r/62484/diff/1-2/
Testing
---
**Unit tests**
Executed existing tests.
**Functional tests**
Executed APIs using CURL calls.
Thanks,
Ashutosh Mestry
esting
---
**Unit tests**
Executed existing tests.
**Functional tests**
Executed APIs using CURL calls.
Thanks,
Ashutosh Mestry
few of existing unit tests.
- Added tests to verify backward compatibility.
**Functional tests**
- Verified regular flow.
- Verified cases:
- Case where server is shutdown before all the parts of a message are
committed.
Thanks,
Ashutosh Mestry
/diff/2-3/
Testing
---
**Unit tests**
No new tests added.
**Functional tests**
Import API tested.
Thanks,
Ashutosh Mestry
(updated)
---
**Unit tests**
- Executed refactored unit tests.
**Functional tests**
- Import API tested.
- Import validated via UI (on multiple browsers).
- Verified abrupt abort scenario.
Thanks,
Ashutosh Mestry
/filters/AtlasKnoxSSOAuthenticationFilter.java
Lines 425 (patched)
<https://reviews.apache.org/r/62382/#comment262352>
You could remove this if you don't have corresponding tests.
- Ashutosh Mestry
On Sept. 22, 2017, 5:32 p.m., Nixon Rodrigues
://reviews.apache.org/r/62797/diff/1-2/
Testing
---
**Unit tests**
- Added existing.
- Executed existing.
**Functional testing**
- Executed tests with large messages.
Thanks,
Ashutosh Mestry
/
Testing
---
**Functional**
- SSL
- Non-SSL
**Unit tests**
- Existing tests.
Thanks,
Ashutosh Mestry
tests with large messages.
Thanks,
Ashutosh Mestry
93d-8f43-5fcaf884c055__create-entity.sh
Thanks,
Ashutosh Mestry
b20ddb13-7300-493d-8f43-5fcaf884c055__create-entity.sh
Thanks,
Ashutosh Mestry
changes in the logs generated.
Thanks,
Ashutosh Mestry
s://reviews.apache.org/r/61665/diff/2-3/
Testing
---
**Unit tests**
Updated unit tests to reproduce the scenarios and verify the fix.
**Functional tests**
Verified regular notification scenarios.
Thanks,
Ashutosh Mestry
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61940/#review184874
---
Ship it!
Ship It!
- Ashutosh Mestry
On Aug. 28, 2017, 1:22
tps://reviews.apache.org/r/61940/#comment261068>
Support _require.js_ with _bust_.
```javascript
```
- Ashutosh Mestry
On Aug. 28, 2017, 1:22 p.m., keval bhatt wrote:
>
> ---
> This is an automatically g
---
**Functional tests**
Observed configuration changes in the logs generated.
Thanks,
Ashutosh Mestry
Diff: https://reviews.apache.org/r/62255/diff/1/
Testing
---
**Unit tests**
No new tests added.
**Functional tests**
Import API tested.
Thanks,
Ashutosh Mestry
unit tests added.
**Functional tests**
- Executed success and failure scenarios.
Thanks,
Ashutosh Mestry
_GraphTransactionInterceptor_ functionality.
- Executed existing unit tests.
**Functional tests**
Import functional tests.
Thanks,
Ashutosh Mestry
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62577/#review186288
---
Ship it!
Ship It!
- Ashutosh Mestry
On Sept. 26, 2017, 3:06
atlas/notification/AtlasNotificationBaseMessage.java
Lines 130 (patched)
<https://reviews.apache.org/r/62681/#comment263314>
Consider encapsulating these methods into separate utility class.
- Ashutosh Mestry
On Sept. 28, 2017, 11:08 p.m.
, non-SSL, Knox integration.
Thanks,
Ashutosh Mestry
/org/apache/atlas/notification/AbstractNotification.java
Lines 204 (patched)
<https://reviews.apache.org/r/62681/#comment263565>
Consider using Math.Ceil.
- Ashutosh Mestry
On Sept. 29, 2017, 10:32 p.m., Madhan Neethiraj
cases:
- Case where server is shutdown before all the parts of a message are
committed.
Thanks,
Ashutosh Mestry
to verify backward compatibility.
**Functional tests**
- Verified regular flow.
- Verified cases:
- Case where server is shutdown before all the parts of a message are
committed.
Thanks,
Ashutosh Mestry
& templates are
loaded from the server for each build that is deployed.
I verified:
- Firefox
- Chrome
Played around with the UI while noting the URLs requested from server.
Good thing is that the _bust_ value remains constant across browsers
(re-stating the point above).
- Ashutosh Me
. Is this intentional? If
yes, what is the rationale?
http://localhost:21000/js/libs/requirejs/require.js
- Ashutosh Mestry
On Aug. 28, 2017, 1:22 p.m., keval bhatt wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit
+1 for the release.
Sarath: Thanks for the efforts on the release!
I have verified the build and artifacts. I did functional testing on search and
Export-Import APIs.
~ ashutosh
Ashutosh Mestry (ames...@apache.org)
...
I get up each morning determined both to change the world and have one
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61552/#review183352
---
Ship it!
Ship It!
- Ashutosh Mestry
On Aug. 11, 2017, 4:25
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61788/#review183357
---
Ship it!
Ship It!
- Ashutosh Mestry
On Aug. 21, 2017, 6:05
)
-
graphdb/titan0/src/main/java/com/thinkaurelius/titan/graphdb/database/idassigner/StandardIDPool.java
PRE-CREATION
Diff: https://reviews.apache.org/r/62009/diff/2/
Changes: https://reviews.apache.org/r/62009/diff/1-2/
Testing
---
Thanks,
Ashutosh Mestry
ATION
repository/src/main/java/org/apache/atlas/repository/util/FilterUtil.java
54d6b40
repository/src/test/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStoreTest.java
728d418
repository/src/test/java/org/apache/atlas/repository/userprofile/UserProfileServiceTest.java
PRE-CREATION
webapp/src/main/java/org/apache/atlas/web/rest/DiscoveryREST.java 52258e3
webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java c32f36e
webapp/src/main/java/org/apache/atlas/web/util/Servlets.java 4a92763
Diff: https://reviews.apache.org/r/62039/diff/2/
Changes: https://reviews.apache.org/r/62039/diff/1-2/
Testing
---
**Deployment**
- Copy _0010-base_model.json_ & _0011-user-profile_model.json_ to server
location: _/usr/hdp/current/atlas-server/models_
- Deploy _atlas.war_
**Unit tests**
Added unit tests for:
- _UserProfileService_.
**Functional testing**
- Verification using CURL calls.
Thanks,
Ashutosh Mestry
ository/src/main/java/org/apache/atlas/repository/util/FilterUtil.java
54d6b40
repository/src/test/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStoreTest.java
728d418
repository/src/test/java/org/apache/atlas/repository/userprofile/UserProfileServiceTest.java
PRE-CREATION
webapp/src/main/java/org/apache/atlas/web/rest/DiscoveryREST.java 52258e3
webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java c32f36e
webapp/src/main/java/org/apache/atlas/web/util/Servlets.java 4a92763
Diff: https://reviews.apache.org/r/62039/diff/1/
Testing
---
**Deployment**
- Copy _0010-base_model.json_ & _0011-user-profile_model.json_ to server
location: _/usr/hdp/current/atlas-server/models_
- Deploy _atlas.war_
**Unit tests**
Added unit tests for:
- _UserProfileService_.
**Functional testing**
- Verification using CURL calls.
Thanks,
Ashutosh Mestry
/StandardIDPool.java
PRE-CREATION
Diff: https://reviews.apache.org/r/62009/diff/1/
Testing
---
Thanks,
Ashutosh Mestry
pache.org/r/61665/diff/4-5/
Testing
---
**Unit tests**
Updated unit tests to reproduce the scenarios and verify the fix.
**Functional tests**
Verified regular notification scenarios.
Thanks,
Ashutosh Mestry
__create-entity.sh
Thanks,
Ashutosh Mestry
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63136/#review189000
---
Ship it!
- Ashutosh Mestry
On Oct. 19, 2017, 4:59 a.m
> On Oct. 23, 2017, 11:08 p.m., Ashutosh Mestry wrote:
> >
I was able to execute ITs and UTs indepdenently.
- Ashutosh
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63136/#rev
t;
> (Updated Oct. 19, 2017, 4:59 a.m.)
>
>
> Review request for atlas, Ashutosh Mestry, Madhan Neethiraj, and Sarath
> Subramanian.
>
>
> Bugs: ATLAS-2218
> https://issues.apache.org/jira/browse/ATLAS-2218
>
>
> Repository: atlas
>
>
> Desc
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63152/#review188996
---
Ship it!
Ship It!
- Ashutosh Mestry
On Oct. 19, 2017, 4:32
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64072/#review191919
---
Ship it!
Ship It!
- Ashutosh Mestry
On Nov. 26, 2017, 7:34
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64048/#review191809
---
Ship it!
Ship It!
- Ashutosh Mestry
On Nov. 23, 2017, 10:24
of tests that exercise the implementation.
- Updated _QueryProcessTest_ to include additional scenarios.
**Functional tests**
- Verified scenarios from UI.
Thanks,
Ashutosh Mestry
(patched)
<https://reviews.apache.org/r/63721/#comment268565>
Where is the file with attributations?
- Ashutosh Mestry
On Nov. 14, 2017, 6:40 a.m., pratik pandey wrote:
>
> ---
> This is an automatically generated e-mail.
up environment to verify various clauses implemented.
**Verification**
Generated Gremlin queries have been verified using JanusGraph shell customized
for Atlas.
Thanks,
Ashutosh Mestry
clauses implemented.
**Functional testing**
- Verified from web UI.
- Verified using '_Saved Search_' feature.
**Additional verification**
Generated Gremlin queries have been verified using JanusGraph shell customized
for Atlas.
Thanks,
Ashutosh Mestry
.
- Verified using '_Saved Search_' feature.
**Additional verification**
Generated Gremlin queries have been verified using JanusGraph shell customized
for Atlas.
Thanks,
Ashutosh Mestry
.
**Functional testing**
- Verified from web UI.
- Verified using '_Saved Search_' feature.
**Additional verification**
Generated Gremlin queries have been verified using JanusGraph shell customized
for Atlas.
Thanks,
Ashutosh Mestry
**
- Verified from web UI.
- Verified using '_Saved Search_' feature.
**Additional verification**
Generated Gremlin queries have been verified using JanusGraph shell customized
for Atlas.
Thanks,
Ashutosh Mestry
d2b3dfdab
Diff: https://reviews.apache.org/r/66899/diff/4/
Changes: https://reviews.apache.org/r/66899/diff/3-4/
Testing
---
**Unit tests**
Additional unit tests to test specific scenarios.
Thanks,
Ashutosh Mestry
tests**
Regular flow verified.
**[Pre-commit
build](https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/404/)**
Thanks,
Ashutosh Mestry
> On April 13, 2018, 12:34 a.m., Ashutosh Mestry wrote:
> > Can you please add instructions on how to use this profile? I tried the
> > patch today and I am still having trouble getting Atlas up.
>
> Pierre Padovani wrote:
> Let me rebase the patch and make some cha
tps://reviews.apache.org/r/66064/#comment284490>
This is getting included even when the profile is not selected.
- Ashutosh Mestry
On May 7, 2018, 9:28 p.m., Pierre Padovani wrote:
>
> ---
> This is an automatically generated e-mail.
, line 163, in
File "./bin/atlas_start.py", line 92, in main
Exception: ('Could not find hbase-site.xml in %s. Please set env var
HBASE_CONF_DIR to the hbase client conf dir',
'./target/apache-atlas-1.0.0-SNAPSHOT-bin/apache-atlas-1.0.0-SNAPSHOT/hbase/conf')
- Ashutosh Mestry
On M
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66064/#review202597
---
Ship it!
Ship It!
- Ashutosh Mestry
On May 7, 2018, 9:28
/TypesStoringEdgeIdsTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/66928/diff/5/
Changes: https://reviews.apache.org/r/66928/diff/4-5/
Testing
---
**Unit tests**
Additional tests added.
**Functional tests**
Regular flow verified.
Thanks,
Ashutosh Mestry
.
**Functional tests**
Regular flow verified.
Thanks,
Ashutosh Mestry
added.
**Functional tests**
Regular flow verified.
**[Pre-commit
build](https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/382/)**
Thanks,
Ashutosh Mestry
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66949/#review202564
---
Ship it!
Ship It!
- Ashutosh Mestry
On May 4, 2018, 10:55
**
Additional tests added.
**Functional tests**
Regular flow verified.
**[Pre-commit
build](https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/423/)**
Thanks,
Ashutosh Mestry
-
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66928/#review203462
-------
On May 17, 2018, 8:48 p.m., Ashutosh Mestry wrote:
>
> ---
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67239/#review203543
---
Ship it!
Ship It!
- Ashutosh Mestry
On May 21, 2018, 10:54
/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/423/)**
Thanks,
Ashutosh Mestry
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67273/#review203719
---
Ship it!
Ship It!
- Ashutosh Mestry
On May 23, 2018, 8:46
/2018/05/24/8230c44b-9f2e-4ae2-bc1a-04424dafc08c__atlas-migration-data.json
Thanks,
Ashutosh Mestry
-data.json
Thanks,
Ashutosh Mestry
and verified the contents.
Thanks,
Ashutosh Mestry
1 - 100 of 1736 matches
Mail list logo