Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Geometry Data Type Returned as Unknown Charset in Results Grid #18630

Open
aborremans opened this issue Mar 4, 2022 · 5 comments · Fixed by #18730
Open

Geometry Data Type Returned as Unknown Charset in Results Grid #18630

aborremans opened this issue Mar 4, 2022 · 5 comments · Fixed by #18730

Comments

@aborremans
Copy link

aborremans commented Mar 4, 2022

  • Azure Data Studio Version: 1.35.0
  • OS Version: macOS Monterey 12.2.1

Steps to Reproduce:

  1. Run the following query in Azure Data Studio

SELECT geometry::STGeomFromText('POINT (-96.70 40.84)',4326) [Geo]
,geometry::STGeomFromText('POINT (-96.70 40.84)',4326).ToString() [GeoString]

Screen Shot 2022-03-04 at 11 56 11 AM

Azure Data Studio Version
Version: 1.35.0
Commit: e9c8eb8
Date: 2022-02-23T17:24:29.469Z
VS Code: 1.59.0
Electron: 13.5.0
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 21.3.0

Does this issue occur when all extensions are disabled?: Yes. This still occurs when all installed extensions are disabled.

@kburtram
Copy link
Member

kburtram commented Mar 7, 2022

I know we don't support spatial data type in ADS but I thought the output was supposed to be the hexadecimal representation of the value. We'll take a look at what's happening here.

@dzsquared
Copy link
Contributor

I've come across this bug now as well. While we don't visualize the spatial data, we absolutely need to be rendering it into the hexadecimal value.

@tdoshin
Copy link

tdoshin commented Mar 9, 2022

Thanks @dzsquared and @kburtram !

@alanrenmsft
Copy link
Contributor

@aborremans Thanks for reporting the issue, a new version with the fix for this issue should be available later this week.

@aborremans
Copy link
Author

@alanrenmsft Great news! Thanks for the quick turnaround on this issue. Much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

7 participants