Yicong-Huang commented on code in PR #8142:
URL: https://github.com/apache/texera/pull/8142#discussion_r3911018776
##########
amber/src/main/scala/org/apache/texera/web/resource/dashboard/user/workflow/WorkflowResource.scala:
##########
@@ -821,6 +821,8 @@ class WorkflowResource extends LazyLogging {
@GET
@Path("/workflow_name")
def getWorkflowName(@QueryParam("wid") wid: Integer): String = {
Review Comment:
`getOwnerName` is the third member of this trio and still carries the defect
this PR fixes in the other two. It has the identical `@QueryParam("wid") wid:
Integer` shape, sits ten lines above the first hunk, and the hub detail page
fetches all three together — so the same omitted parameter still reaches it
unguarded.
Advisory rather than blocking because it is outside the diff, but it is the
cheapest possible follow-up: the same two lines.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
