JS exception on parsing columns with integer col.name

[rgovindji]

Found a bug? Please fill out the sections below... thank you 👍

Issue Summary

In query-result.js line 354 if col.name is returned as a long i.e '404' then exception is thrown.
angular.js:13920 TypeError: n.split is not a function

Steps to Reproduce

1. Make an elastic search query which returns a list of buckets that have integers as the key names
2. When the response is returned, you will see the exception

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?

Technical details:

• Redash Version: v2.0.0
• Browser/OS: Mac / Chrome
• How did you install Redash: Docker

[arikfr]

At first I thought that we should handle this in the client code, but on second thought I think that the column name should be a string and the fix should be in the query runner.

[rgovindji]

The issue we have is that Kibana automatically names this column "2" by default. It's not much of an issue to change it to a string but the issue is not obvious unless you debug it so I can see other users running into this problem often.

…

On Tue, Dec 12, 2017 at 4:36 PM, Arik Fraimovich ***@***.***> wrote: At first I thought that we should handle this in the client code, but on second thought I think that the column name should be a string and the fix should be in the query runner. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1960 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAPWZ8QA56Ermhgb38rqFVjydFb945bpks5s_vHlgaJpZM4PePv-> .