----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51515/#review148723 -----------------------------------------------------------
addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/HiveMetaStoreBridge.java (line 588) <https://reviews.apache.org/r/51515/#comment216225> Can you add tests that column position is set correctly in create table and alter table column reorder? You can add extra validations in existing tests as well - Shwetha GS On Aug. 30, 2016, 6:20 a.m., Sarath Kumar Subramanian wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51515/ > ----------------------------------------------------------- > > (Updated Aug. 30, 2016, 6:20 a.m.) > > > Review request for atlas, Madhan Neethiraj and Suma Shivaprasad. > > > Bugs: ATLAS-772 > https://issues.apache.org/jira/browse/ATLAS-772 > > > Repository: atlas > > > Description > ------- > > When a schema query is issued against a hive table, the hive_column entities > are retrieved from the titan database. Each hive_column entity is a vertex in > titan and currently titan doesn't store any column order position in its > vertex attributes. The hive_columns are retrieved in the sort order of the > Vertex ID which is not same as the Hive column order position. The fix > attached is to introduce a new "position" attribute in hive_column type, > which maintains the column order position. This column order position can be > consumed by any clients - including UI to display the columns in the original > position/order. > > > Diffs > ----- > > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/HiveMetaStoreBridge.java > 1f13d74 > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/model/HiveDataModelGenerator.java > b308cc9 > > Diff: https://reviews.apache.org/r/51515/diff/ > > > Testing > ------- > > Tested the changes with hive column operations – Add new column, Drop column, > Rename column, Move column to first or at a position. > > > Thanks, > > Sarath Kumar Subramanian > >