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

feat: support Synapase Serverless SQL pool #9531

Open
1 task done
ruiyang2015 opened this issue Jul 6, 2024 · 3 comments
Open
1 task done

feat: support Synapase Serverless SQL pool #9531

ruiyang2015 opened this issue Jul 6, 2024 · 3 comments
Labels
feature Features or general enhancements

Comments

@ruiyang2015
Copy link

Is your feature request related to a problem?

Synapse Serverless/Dedicated are both compatible with MS SQL server dialect, but the default behave of Synapse Serveless is tht DBO schema is read only.

c = ibis.mssql.connect(...)
df = pd.DataFrame.from_dict({'v': ['v1', 'v2']})
c.create_table('team', df, database='myschema')

this will getting an error from _register_in_memory_tables, where a temp in memory table is created in the dbo schema even though i tried to specify the database parameter for a different schema.

What is the motivation behind your request?

to support broader ms sql server like server.

Describe the solution you'd like

fix the _register_in_memory_tables code, so it register the in memory temp table in the same schema as the target table.

What version of ibis are you running?

9.1.0

What backend(s) are you using, if any?

synapse serverless

Code of Conduct

  • I agree to follow this project's Code of Conduct
@ruiyang2015 ruiyang2015 added the feature Features or general enhancements label Jul 6, 2024
@deepyaman
Copy link
Contributor

At least for MS SQL, it seems like it was deliberate to create the temp tables under dbo (see #9434), because of some quirks of MS SQL specifically. I think @gforsyth looked into this quite a bit and may be able to chime in here.

That said, what you're saying makes sense, that this approach doesn't work on an implementation where dbo is read-only.

@gforsyth
Copy link
Member

MSSQL doesn't allow you to specify where temp tables are created -- if that behavior is different in Synapase, then the similarity of dialect seems insufficient to allow using the mssql backend there.

@ruiyang2015
Copy link
Author

ruiyang2015 commented Jul 15, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Features or general enhancements
Projects
Status: backlog
Development

No branches or pull requests

3 participants