Skip to content

Commit

Permalink
refactor(oracle): remove cruft
Browse files Browse the repository at this point in the history
  • Loading branch information
gforsyth committed Jan 18, 2024
1 parent 3c2660f commit e98fc64
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 195 deletions.
23 changes: 0 additions & 23 deletions ibis/backends/oracle/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,14 +13,6 @@
import sqlglot as sg
import sqlglot.expressions as sge

Check warning on line 14 in ibis/backends/oracle/__init__.py

View check run for this annotation

Codecov / codecov/patch

ibis/backends/oracle/__init__.py#L14

Added line #L14 was not covered by tests

# Wow, this is truly horrible
# Get out your clippers, it's time to shave a yak.
#
# 1. oracledb is only supported in sqlalchemy 2.0
# 2. Ergo, module hacking is required to avoid doing a silly amount of work
# to create multiple lockfiles or port snowflake away from sqlalchemy
# 3. Also the version needs to be spoofed to be >= 7 or else the cx_Oracle
# dialect barfs
import ibis
import ibis.common.exceptions as exc
import ibis.expr.datatypes as dt
Expand Down Expand Up @@ -487,21 +479,6 @@ def _fetch_from_cursor(self, cursor, schema: sch.Schema) -> pd.DataFrame:
df = OraclePandasData.convert_table(df, schema)
return df

Check warning on line 480 in ibis/backends/oracle/__init__.py

View check run for this annotation

Codecov / codecov/patch

ibis/backends/oracle/__init__.py#L477-L480

Added lines #L477 - L480 were not covered by tests

# def _table_from_schema(
# self,
# name: str,
# schema: sch.Schema,
# temp: bool = False,
# database: str | None = None,
# **kwargs: Any,
# ) -> sa.Table:
# if temp:
# kwargs["oracle_on_commit"] = "PRESERVE ROWS"
# t = super()._table_from_schema(name, schema, temp, database, **kwargs)
# if temp:
# atexit.register(self._clean_up_tmp_table, t)
# return t

def _clean_up_tmp_table(self, name: str) -> None:
with self.begin() as bind:
# global temporary tables cannot be dropped without first truncating them
Expand Down
42 changes: 0 additions & 42 deletions ibis/backends/oracle/datatypes.py

This file was deleted.

130 changes: 0 additions & 130 deletions ibis/backends/oracle/registry.py

This file was deleted.

0 comments on commit e98fc64

Please sign in to comment.