-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61523/
-----------------------------------------------------------
(Updated Aug. 9, 2017, 1:21 p.m.)
Review request for atlas, Apoorv Naik, Ashutosh Mestry, Madhan Neethiraj, Nixon
Rodrigues, and Sarath Subramanian.
Changes
-------
Bug fix
Bugs: ATLAS-2020
https://issues.apache.org/jira/browse/ATLAS-2020
Repository: atlas
Description
-------
1. Searched hive_table in basic search.
2. Checked "Include historical entities" check box - Result table had DELETED
entities.
3. Added Columns filter "columns" - This fired a new query with
excludeDeletedEntities set to True and fetched only the ACTIVE entities and
reset (unchecked) "Include historical entities" .
Expected that the current value of excludeDeletedEntities would be used when
Column filtering.
After Column filtering , explicitly had to check "Include historical entities"
to list the DELETED entities.
Diffs (updated)
-----
dashboardv2/package.json e2e3d8c
dashboardv2/public/css/scss/loader.scss 7d461c8
dashboardv2/public/css/scss/override.scss d2a4bec
dashboardv2/public/css/scss/panel.scss d1c2961
dashboardv2/public/js/router/Router.js f858ee2
dashboardv2/public/js/templates/common/TableLayout_tmpl.html e2b8a86
dashboardv2/public/js/templates/detail_page/DetailPageLayoutView_tmpl.html
3177c7e
dashboardv2/public/js/templates/search/SearchResultLayoutView_tmpl.html
91f1a67
dashboardv2/public/js/utils/CommonViewFunction.js 4abec08
dashboardv2/public/js/utils/TableLayout.js 3c9ff3b
dashboardv2/public/js/views/business_catalog/SideNavLayoutView.js da42554
dashboardv2/public/js/views/common/aboutAtlas.js 9561b83
dashboardv2/public/js/views/detail_page/DetailPageLayoutView.js 56379a4
dashboardv2/public/js/views/entity/EntityDetailTableLayoutView.js e27edf4
dashboardv2/public/js/views/schema/SchemaLayoutView.js 6f9b6e3
dashboardv2/public/js/views/search/SearchDetailLayoutView.js b22fae8
dashboardv2/public/js/views/search/SearchLayoutView.js 0c39b02
dashboardv2/public/js/views/search/SearchResultLayoutView.js d1b1a02
Diff: https://reviews.apache.org/r/61523/diff/2/
Changes: https://reviews.apache.org/r/61523/diff/1-2/
Testing (updated)
-------
Done one round of sanity testing
Thanks,
keval bhatt
