[incubator-pinot] branch removesegfet updated: Removing todo

[je...@apache.org]

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

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


The following commit(s) were added to refs/heads/removesegfet by this push:
     new e405918  Removing todo
e405918 is described below

commit e40591820468d4be099368427ea55a9321e73002
Author: Jennifer Dai <jd...@linkedin.com>
AuthorDate: Thu Jan 31 10:20:04 2019 -0800

    Removing todo
---
 .../org/apache/pinot/server/starter/helix/SegmentFetcherAndLoader.java   | 1 -
 1 file changed, 1 deletion(-)

diff --git a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SegmentFetcherAndLoader.java b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SegmentFetcherAndLoader.java
index 48cdb58..180db4b 100644
--- a/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SegmentFetcherAndLoader.java
+++ b/pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SegmentFetcherAndLoader.java
@@ -199,7 +199,6 @@ public class SegmentFetcherAndLoader {
     try {
       SegmentFetcherFactory.getInstance().getSegmentFetcherBasedOnURI(uri).fetchSegmentToLocal(uri, tempDownloadFile);
       if (crypter != null) {
-        // TODO: We should not need to initialize crypter each time, instead Factory should have an initialized version ready.
         crypter.decrypt(tempDownloadFile, tempTarFile);
       } else {
         tempTarFile = tempDownloadFile;


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