force escape cubeAlias to work with restricted column names

[dgkm]

force escape cubeAlias to work with restricted column names such as "case" in the column name

[paveltiunov]

@dgkmurthy Hey Keshava! Thanks for contributing this one! Just checked: it fails tests. Could you please fix it?

[dgkm]

@dgkmurthy Hey Keshava! Thanks for contributing this one! Just checked: it fails tests. Could you please fix it?

@paveltiunov, I will double check. Any particular test scenarios you are referring to?

[paveltiunov]

@dgkmurthy It's filter join not multiplied and MongoDB tests.

[dgkm]

@dgkmurthy It's filter join not multiplied and MongoDB tests.

@paveltiunov As of now I can only test again Postgres. I have reverted changes and applied to only Postgres Query part. Can you please check?

[paveltiunov]

@dgkmurthy Hey Keshava! Actually your first change was correct. You just need to fix this line: https://github.com/statsbotco/cube.js/blob/d22a809059340e2b7caab7dc8cb122c14e581edd/packages/cubejs-schema-compiler/adapter/BaseQuery.js#L589 There's double escaping.

[dgkm]

@dgkmurthy Hey Keshava! Actually your first change was correct. You just need to fix this line:

https://github.com/statsbotco/cube.js/blob/d22a809059340e2b7caab7dc8cb122c14e581edd/packages/cubejs-schema-compiler/adapter/BaseQuery.js#L589

There's double escaping.

Ok great, can you please check the update now?

[paveltiunov]

@dgkmurthy Looks good! Thanks!