[GitHub] [incubator-druid] capistrant commented on issue #7562: Enable ability to toggle SegmentMetadata request logging on/off

2019-07-16 Thread GitBox
capistrant commented on issue #7562: Enable ability to toggle SegmentMetadata 
request logging on/off
URL: https://github.com/apache/incubator-druid/pull/7562#issuecomment-511932914
 
 
   Thanks all for the tips and info! I'll be cleaning this up this week 
following all the feedback


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:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org



[GitHub] [incubator-druid] capistrant commented on issue #7562: Enable ability to toggle SegmentMetadata request logging on/off

2019-07-12 Thread GitBox
capistrant commented on issue #7562: Enable ability to toggle SegmentMetadata 
request logging on/off
URL: https://github.com/apache/incubator-druid/pull/7562#issuecomment-510985475
 
 
   @leventov That sounds like a good idea to me. How would you propose handling 
erroneous entries by users? Hard failure, Ignore with a warning log, other?


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:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org



[GitHub] [incubator-druid] capistrant commented on issue #7562: Enable ability to toggle SegmentMetadata request logging on/off

2019-07-10 Thread GitBox
capistrant commented on issue #7562: Enable ability to toggle SegmentMetadata 
request logging on/off
URL: https://github.com/apache/incubator-druid/pull/7562#issuecomment-510086321
 
 
   @gianm @Caroline1000 Any interest in reviving this PR to address the related 
issue regarding the logging here?


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:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org