Github user tushargosavi commented on a diff in the pull request:
https://github.com/apache/incubator-apex-core/pull/148#discussion_r47053932
--- Diff:
engine/src/main/java/com/datatorrent/stram/webapp/StramWebServices.java ---
@@ -527,6 +567,33 @@ public JSONObject
getLogicalOperator(@PathParam("operatorName") String operatorN
}
@GET
+ @Path(PATH_LOGICAL_PLAN_MODULES + "/{moduleName:.+}")
--- End diff --
:.+ was required if separator is '/', as internal module names and
operators populated by the module will have '/' in their names.
Current REST api don't handle '/' in names of any component correctly. I
have opened following Jira for it. Will handle this thing from that PR, I will
remove :.+ for now.
https://issues.apache.org/jira/browse/APEXCORE-282
do you see other problem with component having '/' in their names, If thats
the case then I will change the separator. The top voted separator was '/' and
next is '$'.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
