----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73669/#review223687 -----------------------------------------------------------
Fix it, then Ship it! notification/src/main/java/org/apache/atlas/util/CommandHandlerUtility.java Lines 33 (patched) <https://reviews.apache.org/r/73669/#comment312794> +1 'attemptCount' is not a valid member when you are making this a generic method in CommandHandlerUtility use boolean - forceKill - true/false notification/src/main/java/org/apache/atlas/util/CommandHandlerUtility.java Lines 38 (patched) <https://reviews.apache.org/r/73669/#comment312795> getPIDOfPortUse => findProcessIdUsingPort notification/src/main/java/org/apache/atlas/util/CommandHandlerUtility.java Lines 40 (patched) <https://reviews.apache.org/r/73669/#comment312793> 'reader' is never used? consider removing pom.xml Line 1963 (original), 1963 (patched) <https://reviews.apache.org/r/73669/#comment312796> consider using variable here: ${surefire.version} - Sarath Subramanian On Oct. 28, 2021, 2: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, 2: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 > >
