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

[GitBox]

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


With regards,
Apache Git Services