> On Oct. 28, 2021, 9:24 p.m., Ashutosh Mestry wrote: > > notification/src/main/java/org/apache/atlas/util/CommandHandlerUtility.java > > Lines 33 (patched) > > <https://reviews.apache.org/r/73669/diff/2/?file=2254395#file2254395line33> > > > > Change attemptCount to boolean. That way you caller can simply pass > > true or false. > > The logic attemptCount != 0 will depend on caller.
Good catch. Thank you - Sidharth ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73669/#review223683 ----------------------------------------------------------- On Oct. 28, 2021, 9:19 p.m., Sidharth Mishra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73669/ > ----------------------------------------------------------- > > (Updated Oct. 28, 2021, 9:19 p.m.) > > > Review request for atlas, Ashutosh Mestry, Disha Talreja, Radhika Kundam, and > Sarath Subramanian. > > > Bugs: ATLAS-4462 > https://issues.apache.org/jira/browse/ATLAS-4462 > > > Repository: atlas > > > Description > ------- > > ATLAS-4462: Updated the pom to stop jetty before start, Added daemon flag to > true to stop jetty at end of post integration, Added retry logic for Kafka > start having port bind issue, Fixed Infinite loop for Index Recovery Service > > > Diffs > ----- > > addons/falcon-bridge/pom.xml 9d56fd8d3 > addons/hbase-bridge/pom.xml 151a62a77 > addons/hive-bridge/pom.xml f62140918 > addons/impala-bridge/pom.xml 87650e1da > addons/kafka-bridge/pom.xml 2ac19fd20 > addons/sqoop-bridge/pom.xml 03861df84 > addons/storm-bridge/pom.xml 6bfe15b5b > notification/src/main/java/org/apache/atlas/kafka/EmbeddedKafkaServer.java > 19717fb2b > notification/src/main/java/org/apache/atlas/util/CommandHandlerUtility.java > PRE-CREATION > pom.xml 84630e76b > webapp/pom.xml 443559c8d > > > Diff: https://reviews.apache.org/r/73669/diff/2/ > > > Testing > ------- > > Manually tested the Infinite loop issue at Index Recovery Service, Kafka > retry logic for kafka start. Along with this I ran the local tests which > passed. > > > Thanks, > > Sidharth Mishra > >