TEZ-4689: Introduce Node abstraction for DAGAppMaster instead of sepagrate NodeManager-related fields by Aggarwal-Raghav · Pull Request #461 · apache/tez

Aggarwal-Raghav · 2026-02-27T18:42:58Z

No description provided.

Aggarwal-Raghav · 2026-02-27T18:43:32Z

- private final int nmPort;
- private final int nmHttpPort;
+ private String nmHost;
+ private int nmPort;


nmPort is not used anywhere but still keeping getAppNMPort() public api

Aggarwal-Raghav · 2026-02-27T18:46:58Z

+ if (!isLocal) {
+ this.nmHost = nodeContext.getNodeHostString();
+ int nmHttpPort = Integer.parseInt(nodeContext.getNodeHttpPortString());
+ this.containerLogs =


this.containerLogs is making use of nmHttpPort and nmHost, moved it from constructor to serviceInit() and made nmHttpPort as local variable

Aggarwal-Raghav · 2026-02-27T18:52:41Z

 DAGProtos.ConfigurationProto confProto = amExtensions.loadConfigurationProto();
 TezUtilsInternal.addUserSpecifiedTezConfiguration(conf, confProto.getConfKeyValuesList());

+ NodeContext nodeContext = new YarnNodeManagerContext();


I don't think we need to wrap in if statement for checking if framework is yarn. As the YarnNodeManagerContext is using supplier it won't be evaluated immediately and evaluation will happen only it !isLocal.
NOTE: in tez-am docker image tez.local.mode = true

I think we should assume that tez.local.mode is going to become false with this whole initiative eventually, when reaching the optimal state of separate AM and Task containers

abstractdog · 2026-03-04T14:22:58Z

 // Install the tez class loader, which can be used add new resources
 TezClassLoader.setupTezClassLoader();
 Thread.setDefaultUncaughtExceptionHandler(new YarnUncaughtExceptionHandler());
- final String pid = System.getenv().get("JVM_PID");


while I like using ProcessHandle instead of JVM_PID, I'm not sure if this is 100% correct
I'm concerned by the fact that JVM_PID is used in yarn too, see in hadoop code, e.g. https://github.com/apache/hadoop/blob/f49c49aa083b9324ae432902f94a7aaf98f543ea/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapred/YarnChild.java#L126
also, not 100% sure if JVM_PID belongs to the actual Java process id or the launch_container.sh shell script provided by yarn: even if the shell script must be the parent of the jvm process, they are not the same

I'll investigate on this and get back to you on this.

So i tested by putting both
LOG.info("######"); LOG.info("System.getenv().get(\"JVM_PID\") value is: {}", old_pid); LOG.info("ProcessHandle.current().pid() value is : {}", pid); LOG.info("######");
the JVM_PID belongs to launch_container.sh and
ProcessHandler one is DagAppMaster
Screenshot confirming the same.

Will revert this change. Thanks for pointing this!!

abstractdog · 2026-03-04T14:27:57Z

 public int getAppNMPort() {
- return nmPort;
+ return nodeContext.getNodePort();
 }

 public int getAppNMHttpPort() {
- return nmHttpPort;
+ return nodeContext.getNodeHttpPort();


is there a chance we can remove getAppNMPort and getAppNMHttpPort methods altogether?

i thought so but they are public api, but its a tez major version change so we can do so. I wanted to discuss the same with you. Let me know.
there are few variables that can be made final and local to method instead of class varaibles in DagAppMaster. I wanted to clean that up and use JDK 21 features like enhanced switch but the change would have been outside the scope of PR. I can create a separte PR for code hygiene?

code hygiene patches are always welcome
IMO you can remove these methods here, and do more in separate PRs

sure. will update PR shortly and raise separate JIRA/PR for code hygiene. Thanks

tez-yetus · 2026-03-04T20:13:38Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 18s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 4 new or modified test files.
			_ master Compile Tests _
+1 💚	mvninstall	10m 59s		master passed
+1 💚	compile	0m 43s		master passed
+1 💚	checkstyle	1m 6s		master passed
+1 💚	javadoc	0m 36s		master passed
+0 🆗	spotbugs	2m 22s		tez-dag in master has 749 extant spotbugs warnings.
			_ Patch Compile Tests _
+1 💚	mvninstall	0m 31s		the patch passed
+1 💚	codespell	0m 51s		No new issues.
+1 💚	compile	0m 30s		the patch passed
+1 💚	javac	0m 30s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
-0 ⚠️	checkstyle	0m 10s	/buildtool-patch-checkstyle-tez-dag.txt	The patch fails to run checkstyle in tez-dag
+1 💚	javadoc	0m 13s		the patch passed
+1 💚	spotbugs	1m 44s		the patch passed
			_ Other Tests _
+1 💚	unit	5m 58s		tez-dag in the patch passed.
+1 💚	asflicense	0m 13s		The patch does not generate ASF License warnings.
		27m 37s

Subsystem	Report/Notes
Docker	ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/5/artifact/out/Dockerfile
GITHUB PR	#461
Optional Tests	dupname asflicense javac javadoc unit spotbugs checkstyle codespell detsecrets compile
uname	Linux 3e7149cc127d 5.15.0-160-generic #170-Ubuntu SMP Wed Oct 1 10:06:56 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/.yetus/personality.sh
git revision	master / `b882a0c`
Default Java	Ubuntu-21.0.10+7-Ubuntu-124.04
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/5/testReport/
Max. process+thread count	262 (vs. ulimit of 5500)
modules	C: tez-dag U: tez-dag
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/5/console
versions	git=2.43.0 maven=3.8.7 spotbugs=4.9.3 codespell=2.4.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

abstractdog · 2026-03-06T12:22:43Z

hm, how is this possible:
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/5/artifact/out/buildtool-patch-checkstyle-tez-dag.txt

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1:checkstyle (default-cli) on project tez-dag: An error has occurred in Checkstyle report generation. Failed during checkstyle configuration: Exception was thrown while processing /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/tez-dag/src/main/java/org/apache/tez/dag/app/NodeContext.java: IllegalStateException occurred while parsing file /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/tez-dag/src/main/java/org/apache/tez/dag/app/NodeContext.java. /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/tez-dag/src/main/java/org/apache/tez/dag/app/NodeContext.java:18:8: unexpected token: sealed -> [Help 1]

I thought sealed interfaces are present since JDK17, so that must not be a problem...however, it compiled successfully, but the checkstyle:checkstyle target was the one that failed, I hope the same can be reproduced locally

Aggarwal-Raghav · 2026-03-06T12:30:18Z

hm, how is this possible: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/5/artifact/out/buildtool-patch-checkstyle-tez-dag.txt
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1:checkstyle (default-cli) on project tez-dag: An error has occurred in Checkstyle report generation. Failed during checkstyle configuration: Exception was thrown while processing /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/tez-dag/src/main/java/org/apache/tez/dag/app/NodeContext.java: IllegalStateException occurred while parsing file /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/tez-dag/src/main/java/org/apache/tez/dag/app/NodeContext.java. /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/tez-dag/src/main/java/org/apache/tez/dag/app/NodeContext.java:18:8: unexpected token: sealed -> [Help 1]
I thought sealed interfaces are present since JDK17, so that must not be a problem...however, it compiled successfully, but the checkstyle:checkstyle target was the one that failed, I hope the same can be reproduced locally

I'm doing some investigation on checkstyle issue as part of #462 . but using checkstyle 3.6.0 and removing puppycrawler explicit dependency, then its working.

abstractdog · 2026-03-06T12:32:08Z

+ NodeContext nodeContext =
+ (frameworkService instanceof YarnServerFrameworkService)
+ ? new YarnNodeManagerContext()
+ : null;


I think instead of an instanceof check we should let ServerFrameworkService interface handle this: the advantage of separate framework service implementations is to eliminate these type checks (the ZkStandaloneServerFrameworkService can simply return null or a LocalNodeContext with dummy values)

abstractdog · 2026-03-06T12:34:18Z

+ }
+
+ @Override
+ public String getNodeHostString() {


could this be nodeHost? I believe "String" doesn't add any value here, it could be eliminated from variable names as well as interface

abstractdog · 2026-03-06T12:36:01Z

+ private final Supplier<String> nodeHostStringSupplier;
+ private final Supplier<Integer> nodePortStringSupplier;
+ private final Supplier<Integer> nodeHttpPortStringSupplier;


I think we can also remove "Supplier" suffix from the variable names: by looking at the variables, it should be clear that they are Suppliers

abstractdog · 2026-03-06T12:37:08Z

@Aggarwal-Raghav : left some minor comments, but this looks good so far

Aggarwal-Raghav · 2026-03-06T12:40:31Z

@Aggarwal-Raghav : left some minor comments, but this looks good so far

Thanks! Will update PR I'm some time.

tez-yetus · 2026-03-07T08:58:55Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 15s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 4 new or modified test files.
			_ master Compile Tests _
+1 💚	mvninstall	11m 5s		master passed
+1 💚	compile	0m 44s		master passed
+1 💚	checkstyle	1m 2s		master passed
+1 💚	javadoc	0m 37s		master passed
+0 🆗	spotbugs	2m 22s		tez-dag in master has 749 extant spotbugs warnings.
			_ Patch Compile Tests _
+1 💚	mvninstall	0m 31s		the patch passed
+1 💚	codespell	0m 52s		No new issues.
+1 💚	compile	0m 32s		the patch passed
+1 💚	javac	0m 32s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
-0 ⚠️	checkstyle	0m 11s	/buildtool-patch-checkstyle-tez-dag.txt	The patch fails to run checkstyle in tez-dag
+1 💚	javadoc	0m 13s		the patch passed
+1 💚	spotbugs	1m 41s		the patch passed
			_ Other Tests _
+1 💚	unit	6m 2s		tez-dag in the patch passed.
+1 💚	asflicense	0m 13s		The patch does not generate ASF License warnings.
		27m 41s

Subsystem	Report/Notes
Docker	ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/6/artifact/out/Dockerfile
GITHUB PR	#461
Optional Tests	dupname asflicense javac javadoc unit spotbugs checkstyle codespell detsecrets compile
uname	Linux b39160e6b376 5.15.0-160-generic #170-Ubuntu SMP Wed Oct 1 10:06:56 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/.yetus/personality.sh
git revision	master / `86675af`
Default Java	Ubuntu-21.0.10+7-Ubuntu-124.04
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/6/testReport/
Max. process+thread count	217 (vs. ulimit of 5500)
modules	C: tez-dag U: tez-dag
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/6/console
versions	git=2.43.0 maven=3.8.7 spotbugs=4.9.3 codespell=2.4.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

abstractdog · 2026-03-09T10:33:24Z

@Aggarwal-Raghav : this patch looks good to me, I'm just wondering if you found the root cause of the checkstyle problem and it can be proved that it's going to go away soon and precommit remains in a good shape

Aggarwal-Raghav · 2026-03-09T10:46:27Z

@Aggarwal-Raghav : this patch looks good to me, I'm just wondering if you found the root cause of the checkstyle problem and it can be proved that it's going to go away soon and precommit remains in a good shape

The puppycrawler checkstyle 8.35 (Jul 26, 2020) is before jdk17 (September 14, 2021) was launched, hence this error. Upgrading checkstyle version to 3.6.0 will help.

abstractdog · 2026-03-09T10:49:10Z

@Aggarwal-Raghav : this patch looks good to me, I'm just wondering if you found the root cause of the checkstyle problem and it can be proved that it's going to go away soon and precommit remains in a good shape
The puppycrawler checkstyle 8.35 (Jul 26, 2020) is before jdk17 (September 14, 2021) was launched, hence this error. Upgrading checkstyle version to 3.6.0 will help.

thanks, good to know! can you create a Jira + PR about that to unblock this one? that would help a lot in modernizing Tez precommit infrastructure with a simple patch

Aggarwal-Raghav · 2026-03-09T10:50:42Z

@Aggarwal-Raghav : this patch looks good to me, I'm just wondering if you found the root cause of the checkstyle problem and it can be proved that it's going to go away soon and precommit remains in a good shape
The puppycrawler checkstyle 8.35 (Jul 26, 2020) is before jdk17 (September 14, 2021) was launched, hence this error. Upgrading checkstyle version to 3.6.0 will help.
thanks, good to know! can you create a Jira + PR about that to unblock this one? that would help a lot in modernizing Tez precommit infrastructure with a simple patch

sure.

…rate NodeManager-related fields

tez-yetus · 2026-03-11T11:55:33Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	7m 58s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 4 new or modified test files.
			_ master Compile Tests _
+1 💚	mvninstall	10m 19s		master passed
+1 💚	compile	0m 44s		master passed
+1 💚	checkstyle	1m 27s		master passed
+1 💚	javadoc	0m 37s		master passed
+0 🆗	spotbugs	2m 1s		tez-dag in master has 749 extant spotbugs warnings.
			_ Patch Compile Tests _
+1 💚	mvninstall	0m 31s		the patch passed
+1 💚	codespell	0m 51s		No new issues.
+1 💚	compile	0m 30s		the patch passed
+1 💚	javac	0m 30s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
-0 ⚠️	checkstyle	0m 24s	/results-checkstyle-tez-dag.txt	tez-dag: The patch generated 10 new + 278 unchanged - 5 fixed = 288 total (was 283)
+1 💚	javadoc	0m 13s		the patch passed
+1 💚	spotbugs	1m 44s		the patch passed
			_ Other Tests _
+1 💚	unit	5m 35s		tez-dag in the patch passed.
+1 💚	asflicense	0m 13s		The patch does not generate ASF License warnings.
		34m 29s

Subsystem	Report/Notes
Docker	ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/7/artifact/out/Dockerfile
GITHUB PR	#461
Optional Tests	dupname asflicense javac javadoc unit spotbugs checkstyle codespell detsecrets compile
uname	Linux 53e0dbc7b32b 5.15.0-160-generic #170-Ubuntu SMP Wed Oct 1 10:06:56 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/.yetus/personality.sh
git revision	master / `8203171`
Default Java	Ubuntu-21.0.10+7-Ubuntu-124.04
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/7/testReport/
Max. process+thread count	231 (vs. ulimit of 5500)
modules	C: tez-dag U: tez-dag
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/7/console
versions	git=2.43.0 maven=3.8.7 spotbugs=4.9.3 codespell=2.4.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

Aggarwal-Raghav · 2026-03-11T11:58:01Z

@abstractdog , this is ready. checkstyle is running now.

abstractdog · 2026-03-11T12:00:44Z

@abstractdog , this is ready. checkstyle is running now.

https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/7/artifact/out/results-checkstyle-tez-dag.txt
"Line is longer " kind of alerts are new, can you please fix them? the rest looks okay, e.g. ParameterNumber was just improved here, because you replaced 3 params with a context

Aggarwal-Raghav · 2026-03-11T12:02:53Z

@abstractdog , this is ready. checkstyle is running now.
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/7/artifact/out/results-checkstyle-tez-dag.txt "Line is longer " kind of alerts are new, can you please fix them? the rest looks okay, e.g. ParameterNumber was just improved here, because you replaced 3 params with a context

Will use intelliJ-google-java-format. Initially, I didn't do it because it is causing diff issue in github because of formatting, making difficult to review.

abstractdog

LGTM, pending tests

tez-yetus · 2026-03-11T12:45:44Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 16s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 4 new or modified test files.
			_ master Compile Tests _
+1 💚	mvninstall	10m 22s		master passed
+1 💚	compile	0m 45s		master passed
+1 💚	checkstyle	1m 24s		master passed
+1 💚	javadoc	0m 35s		master passed
+0 🆗	spotbugs	2m 4s		tez-dag in master has 749 extant spotbugs warnings.
			_ Patch Compile Tests _
+1 💚	mvninstall	0m 32s		the patch passed
+1 💚	codespell	0m 51s		No new issues.
+1 💚	compile	0m 31s		the patch passed
+1 💚	javac	0m 31s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
-0 ⚠️	checkstyle	0m 24s	/results-checkstyle-tez-dag.txt	tez-dag: The patch generated 2 new + 279 unchanged - 4 fixed = 281 total (was 283)
+1 💚	javadoc	0m 14s		the patch passed
+1 💚	spotbugs	1m 43s		the patch passed
			_ Other Tests _
+1 💚	unit	6m 21s		tez-dag in the patch passed.
+1 💚	asflicense	0m 14s		The patch does not generate ASF License warnings.
		27m 36s

Subsystem	Report/Notes
Docker	ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/8/artifact/out/Dockerfile
GITHUB PR	#461
Optional Tests	dupname asflicense javac javadoc unit spotbugs checkstyle codespell detsecrets compile
uname	Linux ceebfe522922 5.15.0-160-generic #170-Ubuntu SMP Wed Oct 1 10:06:56 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-461/src/.yetus/personality.sh
git revision	master / `c6aaa2f`
Default Java	Ubuntu-21.0.10+7-Ubuntu-124.04
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/8/testReport/
Max. process+thread count	249 (vs. ulimit of 5500)
modules	C: tez-dag U: tez-dag
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/8/console
versions	git=2.43.0 maven=3.8.7 spotbugs=4.9.3 codespell=2.4.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

Aggarwal-Raghav · 2026-03-12T16:16:17Z

is there any way to re-trigger the Github Actions without pushing the changes again?

abstractdog · 2026-03-13T07:23:49Z

is there any way to re-trigger the Github Actions without pushing the changes again?

I just retriggered them

Aggarwal-Raghav commented Feb 27, 2026
View reviewed changes

Aggarwal-Raghav force-pushed the TEZ-4689 branch from c7923b1 to fca17d7Compare February 27, 2026 18:49

Aggarwal-Raghav commented Feb 27, 2026
View reviewed changes

This comment was marked as outdated.
Sign in to view

Aggarwal-Raghav commented Feb 28, 2026
View reviewed changes

Comment threadtez-dag/src/main/java/org/apache/tez/dag/app/DAGAppMaster.java Outdated

Aggarwal-Raghav force-pushed the TEZ-4689 branch from fca17d7 to 5e12682Compare February 28, 2026 17:12

Aggarwal-Raghav marked this pull request as ready for review February 28, 2026 17:13

This comment was marked as outdated.
Sign in to view

abstractdog reviewed Mar 3, 2026
View reviewed changes

Comment threadtez-dag/src/main/java/org/apache/tez/dag/app/LocalNodeContext.java Outdated

abstractdog reviewed Mar 3, 2026
View reviewed changes

Comment threadtez-dag/src/main/java/org/apache/tez/dag/app/DAGAppMaster.java Outdated

This comment was marked as outdated.
Sign in to view

abstractdog reviewed Mar 4, 2026
View reviewed changes

Aggarwal-Raghav requested a review from abstractdog March 5, 2026 17:31

abstractdog reviewed Mar 6, 2026
View reviewed changes

TEZ-4689: Introduce Node abstraction for DAGAppMaster instead of sepa…
08c0fcc
…rate NodeManager-related fields

Aggarwal-Raghav added 3 commits March 11, 2026 16:49

Review comments
5dfb805

Review comments 2
e84f84b

review comments 3
8203171

Aggarwal-Raghav force-pushed the TEZ-4689 branch from 86675af to 8203171Compare March 11, 2026 11:20

Checkstyle issue fix
c6aaa2f

abstractdog self-requested a review March 11, 2026 12:31

abstractdog approved these changes Mar 11, 2026
View reviewed changes

abstractdog merged commit 9725e38 into apache:masterMar 13, 2026
5 of 12 checks passed

Uh oh!

Conversation

Aggarwal-Raghav commented Feb 27, 2026

Uh oh!

Aggarwal-RaghavFeb 27, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Aggarwal-RaghavFeb 27, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment was marked as outdated.

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

Uh oh!

This comment was marked as outdated.

abstractdogMar 4, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Aggarwal-RaghavMar 4, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

abstractdogMar 4, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tez-yetus commented Mar 4, 2026

Uh oh!

abstractdog commented Mar 6, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Aggarwal-Raghav commented Mar 6, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

abstractdog commented Mar 6, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Aggarwal-Raghav commented Mar 6, 2026

Uh oh!

tez-yetus commented Mar 7, 2026

Uh oh!

abstractdog commented Mar 9, 2026

Uh oh!

Aggarwal-Raghav commented Mar 9, 2026

Uh oh!

abstractdog commented Mar 9, 2026

Uh oh!

Aggarwal-Raghav commented Mar 9, 2026

Uh oh!

tez-yetus commented Mar 11, 2026

Uh oh!

Aggarwal-Raghav commented Mar 11, 2026

Uh oh!

abstractdog commented Mar 11, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Aggarwal-Raghav commented Mar 11, 2026

Uh oh!

Aggarwal-RaghavFeb 27, 2026•
edited
Loading

Aggarwal-RaghavFeb 27, 2026•
edited
Loading

abstractdogMar 4, 2026•
edited
Loading

Aggarwal-RaghavMar 4, 2026•
edited
Loading

abstractdogMar 4, 2026•
edited
Loading

abstractdog commented Mar 6, 2026•
edited
Loading

abstractdog commented Mar 6, 2026•
edited
Loading

abstractdog commented Mar 11, 2026•
edited
Loading