komsit37 / sublime-q

Sublime Text Plugin for q/kdb
MIT License
24 stars 9 forks source link

large number return causes kdb 3.6 to crash #17

Closed pindash closed 5 years ago

pindash commented 5 years ago

60000000000 in sublime causes q to crash with wsfull

I'm using sublime-q version 0.2.11

KDB version 3.6 2018.07.30 on red hat 6.6

hongshaoyang commented 5 years ago

+1. reproducible on:

Does not occur if using qpad 1.8

komsit37 commented 5 years ago

@pindash @hongshaoyang Thanks for reporting.

This is caused by my custom q code wrapper which tries to figure out table dimension from the result. In short, it tries to to execute cols 60000000000 and got 'wsfull. The code is in this line: https://github.com/komsit37/sublime-q/blob/master/q_send.py#L47

Calling cols on non table-like data (i.e. long) will raise error (so I wrap it in trap), but it shouldn't cause 'wsfull. Anyway, I will find workaround for this.

komsit37 commented 5 years ago

I've released the fix. Package Control should automatically pickup the latest version soon.

pindash commented 5 years ago

Thank you. I had been using your plugin for a long time without issue but always with 3.4, so that makes total sense.

On Thu, Nov 1, 2018, 00:14 komsit37 <notifications@github.com wrote:

Closed #17 https://github.com/komsit37/sublime-q/issues/17.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/komsit37/sublime-q/issues/17#event-1939588492, or mute the thread https://github.com/notifications/unsubscribe-auth/ACDWYQTBfj1ov_zUDSoVcUiu5el2D7luks5uqnUNgaJpZM4X5J-0 .