[GitHub] ihuzenko removed a comment on issue #1608: DRILL-6960: Auto Limit Wrapping should not apply to non-select query

[GitBox <gi...@apache.org>]

ihuzenko removed a comment on issue #1608: DRILL-6960: Auto Limit Wrapping should not apply to non-select query
URL: https://github.com/apache/drill/pull/1608#issuecomment-458127161
 
 
   Hello @kkhatua , sorry for the late answer. I wanted to start review from check that fix works fine, but suddenly bumped into an issue. I wasn't able to submit query with my current locale, because it affects number formatting. 
   
   1. Tried to use default number: 
   ![drill-6960_locale_error](https://user-images.githubusercontent.com/4282630/51838059-ef6dac80-230d-11e9-9a8e-354a8361b971.png)
   
   2. Tried to remove comma from default number: 
   ![drill-6960_locale_error2](https://user-images.githubusercontent.com/4282630/51838111-11ffc580-230e-11e9-9698-520da447a278.png)
   
   3. Tried to use own auto limit 50 :
   ![drill-6960_default_number_taken_instead](https://user-images.githubusercontent.com/4282630/51838150-2a6fe000-230e-11e9-9335-717d49cad563.png)
   
   After those failed attempts I executed the line ```console.log(window.navigator.userLanguage || window.navigator.language)``` in JS console and got locale ```en-US``` in my chromium-browser. 
   
   @kkhatua, can you please fix the blocking issue in order to unlock further review ? Thank you in advance, Igor. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services