You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2021/08/23 11:10:56 UTC

[GitHub] [pinot] cxing opened a new issue #7348: Minion will throw null point exception if protocol is null

cxing opened a new issue #7348:
URL: https://github.com/apache/pinot/issues/7348


   Need fix code:
   https://github.com/apache/pinot/blob/master/pinot-common/src/main/java/org/apache/pinot/common/utils/fetcher/SegmentFetcherFactory.java#L119
   
   ![image](https://user-images.githubusercontent.com/13634486/130437879-67189dec-ee44-4d61-9552-29e3edd95420.png)
   


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

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

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



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


[GitHub] [pinot] cxing commented on issue #7348: Minion will throw null point exception if protocol is null

Posted by GitBox <gi...@apache.org>.
cxing commented on issue #7348:
URL: https://github.com/apache/pinot/issues/7348#issuecomment-905117884


   > @cxing Thanks for reporting the issue. I believe this is caused by peer download where the download url is not set in the segment ZK metadata. Please verify if this is the case and we can make the fix properly
   
   Hi @Jackie-Jiang Yes it caused by the download url is not set in the segment ZK metadata and i think that can can add null point check or it will throw Null point exception


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

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

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



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


[GitHub] [pinot] Jackie-Jiang commented on issue #7348: Minion will throw null point exception if protocol is null

Posted by GitBox <gi...@apache.org>.
Jackie-Jiang commented on issue #7348:
URL: https://github.com/apache/pinot/issues/7348#issuecomment-903977538


   @cxing Thanks for reporting the issue. I believe this is caused by peer download where the download url is not set in the segment ZK metadata. Please verify if this is the case and we can make the fix properly


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

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

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



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


[GitHub] [pinot] cxing edited a comment on issue #7348: Minion will throw null point exception if protocol is null

Posted by GitBox <gi...@apache.org>.
cxing edited a comment on issue #7348:
URL: https://github.com/apache/pinot/issues/7348#issuecomment-905117884


   > @cxing Thanks for reporting the issue. I believe this is caused by peer download where the download url is not set in the segment ZK metadata. Please verify if this is the case and we can make the fix properly
   
   Hi @Jackie-Jiang Yes it's caused by the download url is not set in the segment ZK metadata, I think that can add null point check or it will throw null point exception


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

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

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



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