Skip to content

Commit

Permalink
Clean up remaining issues with hyperd.log generation
Browse files Browse the repository at this point in the history
  • Loading branch information
WillAyd committed Aug 21, 2024
1 parent 88f940c commit fa79af6
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/pantab/_reader.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,8 @@ def frames_from_hyper(

table_names = []
with tempfile.TemporaryDirectory() as tmp_dir, tab_api.HyperProcess(
tab_api.Telemetry.DO_NOT_SEND_USAGE_DATA_TO_TABLEAU
tab_api.Telemetry.DO_NOT_SEND_USAGE_DATA_TO_TABLEAU,
parameters={"log_config": ""},
) as hpe:
tmp_db = shutil.copy(source, tmp_dir)
with tab_api.Connection(hpe.endpoint, tmp_db) as connection:
Expand Down
2 changes: 1 addition & 1 deletion src/pantab/reader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -437,7 +437,7 @@ static auto ReleaseArrowStream(void *ptr) noexcept -> void {
auto read_from_hyper_query(const std::string &path, const std::string &query)
-> nb::capsule {
const std::unordered_map<std::string, std::string> params = {
{"log_config", ""}};
{"log_config", ""}, {"default_database_version", "4"}};
const hyperapi::HyperProcess hyper{
hyperapi::Telemetry::DoNotSendUsageDataToTableau, "", std::move(params)};
hyperapi::Connection connection(hyper.getEndpoint(), path);
Expand Down
3 changes: 2 additions & 1 deletion tests/test_writer.py
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,8 @@ def test_can_write_wkt_as_geo(tmp_hyper):

pt.frame_to_hyper(df, tmp_hyper, table="test", geo_columns=["geography"])
with tab_api.HyperProcess(
tab_api.Telemetry.DO_NOT_SEND_USAGE_DATA_TO_TABLEAU
tab_api.Telemetry.DO_NOT_SEND_USAGE_DATA_TO_TABLEAU,
parameters={"log_config": ""},
) as hyper:
with tab_api.Connection(
hyper.endpoint, tmp_hyper, tab_api.CreateMode.CREATE_IF_NOT_EXISTS
Expand Down

0 comments on commit fa79af6

Please sign in to comment.