----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62874/#review187620 -----------------------------------------------------------
addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java Line 579 (original), 579 (patched) <https://reviews.apache.org/r/62874/#comment264659> entities will have a maximum of 2 entities here, hence it is okay to batch them in a single message. - Madhan Neethiraj On Oct. 11, 2017, 1:50 a.m., Apoorv Naik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62874/ > ----------------------------------------------------------- > > (Updated Oct. 11, 2017, 1:50 a.m.) > > > Review request for atlas, Ashutosh Mestry, Madhan Neethiraj, and Sarath > Subramanian. > > > Bugs: ATLAS-2198 > https://issues.apache.org/jira/browse/ATLAS-2198 > > > Repository: atlas > > > Description > ------- > > For the reported use case, Hive hook messages contains around 508 entities in > a single message. > This causes the Base64 encoder to fail while memory allocation and the entire > message gets dropped because the hive process dies. > > This fix splits one giant message for all entity into N small messages per > entity (Referenceable) > > > Diffs > ----- > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java > aca56454 > > > Diff: https://reviews.apache.org/r/62874/diff/2/ > > > Testing > ------- > > Fired reported command via beeline and verified "view" and it's col lineage > shows up in Atlas UI. > > > Thanks, > > Apoorv Naik > >
