[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5459: [Cleanup] Merge RealtimeSegmentOnlineOfflineStateModel and SegmentOnlineOfflineStateModel in CommonConstants

2020-05-28 Thread GitBox
Jackie-Jiang commented on a change in pull request #5459: URL: https://github.com/apache/incubator-pinot/pull/5459#discussion_r432246733 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/routing/RoutingManager.java ## @@ -51,7 +51,7 @@ import

[GitHub] [incubator-pinot] Jackie-Jiang commented on a change in pull request #5459: [Cleanup] Merge RealtimeSegmentOnlineOfflineStateModel and SegmentOnlineOfflineStateModel in CommonConstants

2020-05-28 Thread GitBox
Jackie-Jiang commented on a change in pull request #5459: URL: https://github.com/apache/incubator-pinot/pull/5459#discussion_r432207006 ## File path: pinot-broker/src/main/java/org/apache/pinot/broker/routing/RoutingManager.java ## @@ -51,7 +51,7 @@ import