You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ne...@apache.org on 2019/01/11 18:17:31 UTC

[incubator-pinot] branch dummy_commit created (now fc6368a)

This is an automated email from the ASF dual-hosted git repository.

nehapawar pushed a change to branch dummy_commit
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git.


      at fc6368a  Change javadoc in RealtimeSegmentRelocator

This branch includes the following new commits:

     new fc6368a  Change javadoc in RealtimeSegmentRelocator

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[incubator-pinot] 01/01: Change javadoc in RealtimeSegmentRelocator

Posted by ne...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

nehapawar pushed a commit to branch dummy_commit
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git

commit fc6368a65ec11821096325197cbfc645c6fe7c89
Author: Neha Pawar <np...@linkedin.com>
AuthorDate: Fri Jan 11 10:17:10 2019 -0800

    Change javadoc in RealtimeSegmentRelocator
---
 .../controller/helix/core/relocation/RealtimeSegmentRelocator.java   | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/relocation/RealtimeSegmentRelocator.java b/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/relocation/RealtimeSegmentRelocator.java
index cce9cd5..fc8ec51 100644
--- a/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/relocation/RealtimeSegmentRelocator.java
+++ b/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/relocation/RealtimeSegmentRelocator.java
@@ -45,9 +45,8 @@ import org.slf4j.LoggerFactory;
 
 
 /**
- * Manager to relocate completed segments to completed servers
- * Segment relocation will be done by this manager, instead of directly moving segments to completed servers on completion,
- * so that we don't get segment downtime when a segment is in transition
+ * Manager to relocate completed segments to "completed" servers
+ * We don't move segments directly to completed servers on completion, because we want to avoid a downtime when a segment is in transition
  *
  * We only relocate segments for realtime tables, and only if tenant config indicates that relocation is required
  * A segment will be relocated, one replica at a time, once all of its replicas are in ONLINE state and all/some are on servers other than completed servers


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org