microsoft / jupyter-Kqlmagic

Extension (Magic) to Jupyter notebook and Jupyter lab, that enable notebook experience working with Kusto, ApplicationInsights, and LogAnalytics data.
Other
85 stars 31 forks source link

Parsing fails when trying to connect to database with space in name #9

Closed joslevin closed 5 years ago

joslevin commented 5 years ago

Trying to connect to a database that has a space in the name fails. Example:

%kql azuredataexplorer://code;cluster='my-cluster';database='Database WithSpaceInName';Alias='IDontCare'

returns failed to set database, due to invalid str value 'Database.

Note that connecting to this database works correctly in other applications. If I have some free bandwidth, I'll try to dig into why this happens - but I'd just like to bring it to your attention now.

mbnshtck commented 5 years ago

Thanks, for letting me know, will create a fix, and let you know.

Michael


From: joslevin notifications@github.com Sent: Friday, March 15, 2019 11:53:12 PM To: Microsoft/jupyter-Kqlmagic Cc: Subscribed Subject: [Microsoft/jupyter-Kqlmagic] Parsing fails when trying to connect to database with space in name (#9)

Trying to connect to a database that has a space in the name fails. Example:

%kql azuredataexplorer://code;cluster='my-cluster';database='Database WithSpaceInName';Alias='IDontCare'

returns failed to set database, due to invalid str value 'Database.

Note that connecting to this database works correctly in other applications. If I have some free bandwidth, I'll try to dig into why this happens - but I'd just like to bring it to your attention now.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fjupyter-Kqlmagic%2Fissues%2F9&data=02%7C01%7Cmichabin%40microsoft.com%7C7928a790e6e04bf225fe08d6a9909f3b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636882835944937667&sdata=5VqvNXfz7r%2FaNFfYyycJb%2B1FGdAaKUKVcT2Dg19wGOU%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIyl3hDVy-eyPDhtNGWHuPHxT6mlM6KWks5vXBZIgaJpZM4b3XQW&data=02%7C01%7Cmichabin%40microsoft.com%7C7928a790e6e04bf225fe08d6a9909f3b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636882835944937667&sdata=RmHio8xwXwQSi5W0GDEX8hMnm1yWPgRt%2BtOIah4ChYA%3D&reserved=0.

mbnshtck commented 5 years ago

Fixed