You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2021/01/19 14:56:46 UTC

[GitHub] [hudi] yanghua edited a comment on pull request #2458: [MINOR] Rename FileSystemViewHandler to Router and corrected the class comment

yanghua edited a comment on pull request #2458:
URL: https://github.com/apache/hudi/pull/2458#issuecomment-762893505


   > @yanghua Sorry that I am not fully get the point here. Handler should be ok here?
   
   It should be a router to route all the request uri to the mapped handlers. Here is the registry: https://github.com/apache/hudi/blob/c2023342eb92c0f54b4bb958a6284e065eadc8bc/hudi-timeline-service/src/main/java/org/apache/hudi/timeline/service/Router.java#L74
   
   IMO, the real handlers all extend `Handler`.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org