Skip to content

load_edges_from_file parser error #2151

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

Open
roamerop opened this issue Feb 10, 2025 · 4 comments
Open

load_edges_from_file parser error #2151

roamerop opened this issue Feb 10, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@roamerop
Copy link

load_edges_from_file parser error
I follow the document https://age.apache.org/age-manual/master/intro/agload.html to import data from file. After imported, the edge table has city properties filed import unnecessary csv value(end_vertex_type), as the following image shows.

Image

How are you accessing AGE (Command line, driver, etc.)?

  • [Navicate]

Expected behavior
The end_vertex_type should not import to properties filed.

Environment (please complete the following information):

  • Version: [docker.m.daocloud.io/apache/age] PostgreSQL 16.3 (Debian 16.3-1.pgdg120+1) on x86_64-pc-linux-gnu, compiled by gcc (Debian 12.2.0-14) 12.2.0, 64-bit
@roamerop roamerop added the bug Something isn't working label Feb 10, 2025
@sorrell
Copy link
Contributor

sorrell commented Feb 13, 2025

I can confirm and am able to replicate this.

@sorrell
Copy link
Contributor

sorrell commented Feb 13, 2025

Looks like we should be going from start_index + 1 here. Have confirmed that fixes this locally.

for (i = start_index; i < fields_len; i++)

@jrgemignani
Copy link
Contributor

@sorrell We should probably have @MuhammadTahaNaveed look into this, if he is available and has time to do so. He was the last to work on the loader, and there were significant changes #2044

@MuhammadTahaNaveed
Copy link
Member

MuhammadTahaNaveed commented Feb 23, 2025

Hi @roamerop @sorrell

By default, the Docker image with the latest tag is pulled, which corresponds to AGE version 1.5.0. I was able to reproduce the issue on AGE 1.5.0 but not on the master branch, which includes many fixes made after the 1.5.0 release, including #2044 for the csv loader.

You can use the dev_snapshot_PG<version> tag, which contains all the fixes made after 1.5.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants