Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

> On April 4, 2018, 12:34 a.m., Vihang Karajgaonkar wrote: > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java > > Lines 408 (patched) > > > > > > why do we need !success che

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

> On April 4, 2018, 12:34 a.m., Vihang Karajgaonkar wrote: > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java > > Lines 408 (patched) > > > > > > why do we need !success che

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65985/ --- (Updated April 4, 2018, 4:06 p.m.) Review request for hive, Alexander Kolbasov,

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

> On April 4, 2018, 12:34 a.m., Vihang Karajgaonkar wrote: > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java > > Lines 408 (patched) > > > > > > why do we need !success che

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65985/#review200405 --- Fix it, then Ship it! LGTM. Just one minor comment below. sta

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65985/ --- (Updated March 15, 2018, 5:45 p.m.) Review request for hive, Alexander Kolbasov

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

> On March 12, 2018, 4:45 p.m., Na Li wrote: > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java > > Lines 372 (patched) > > > > > > why when db name is changed at alter tabl

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

> On March 13, 2018, 3:35 a.m., Alexander Kolbasov wrote: > > standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java > > Lines 362 (patched) > > > > > > Would it make sense to move

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65985/#review199063 --- standalone-metastore/src/main/java/org/apache/hadoop/hive/metasto

Re: Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65985/#review199027 --- standalone-metastore/src/main/java/org/apache/hadoop/hive/metasto

Review Request 65985: HIVE-18783: ALTER TABLE post-commit listener does not include the transactional listener responses

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65985/ --- Review request for hive, Alexander Kolbasov, Sahil Takiar, and Vihang Karajgaonk