Skip to content

Conversation

@ckunki
Copy link
Contributor

@ckunki ckunki commented Nov 28, 2025

Closes #440
Closes #441

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@ckunki ckunki temporarily deployed to approve-test-execution November 28, 2025 07:23 — with GitHub Actions Inactive
@ckunki ckunki requested a deployment to approve-gpu-test-execution November 28, 2025 07:23 — with GitHub Actions Waiting
@ckunki ckunki temporarily deployed to approve-test-execution November 28, 2025 07:58 — with GitHub Actions Inactive
@ckunki ckunki requested a deployment to approve-gpu-test-execution November 28, 2025 07:58 — with GitHub Actions Waiting
@ckunki
Copy link
Contributor Author

ckunki commented Nov 28, 2025

Broken notebook test:

  • Cell In[19], line 12 (Cell Index starts a 2 apparently)
  • ETL-2210: AWS URL is invalid: Provided URL is not a valid AWS S3 URL(AI_LAB_FIRST_STEPS_S3/first_steps/US_FLIGHTS_FEB_2024.parquet).
  • Build logs

source code

from exasol.nb_connector.connections import open_pyexasol_connection

query = "IMPORT INTO {table!q} FROM PARQUET AT {connection} FILE {file!s}"

params = {
    "table": (ai_lab_config.db_schema, "US_FLIGHTS"),
    "connection": "AI_LAB_FIRST_STEPS_S3",
    "file": "first_steps/US_FLIGHTS_FEB_2024.parquet",
}

with open_pyexasol_connection(ai_lab_config, compression=True) as conn:
    result = conn.execute(query, params)
print(f"Imported {result.rowcount()} rows.")

Assumed mistake: prepared statement needs to mention connection with formatting !q or !r.

ArBridgeman
ArBridgeman previously approved these changes Dec 3, 2025
@ckunki ckunki dismissed stale reviews from ArBridgeman and tkilias via 479e5f9 December 4, 2025 14:37
@ckunki ckunki temporarily deployed to approve-test-execution December 4, 2025 14:37 — with GitHub Actions Inactive
@ckunki ckunki temporarily deployed to approve-gpu-test-execution December 4, 2025 14:37 — with GitHub Actions Inactive
@ckunki ckunki merged commit dcaf515 into main Dec 5, 2025
29 of 32 checks passed
@ckunki ckunki deleted the feature/440-Re-enabled_Parquet_import_in_notebook_first_steps.ipynb branch December 5, 2025 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

User Guide: Add section "Getting Started" Re-enable parquet import in notebook first_steps.ipynb

4 participants