postgraph icon indicating copy to clipboard operation
postgraph copied to clipboard

Query: building failed by undeclared variables in cypher_analyze.c

Open xiechuanjie opened this issue 1 year ago • 6 comments

What is your question?

I am evaluating the possibility to extend the PolyhedralSurface geometry type in postgraph, in order to store BIM (building Information modeling) graph. But source code make failed by undeclared variables in cypher_analyze.c, The erros are "parse_hook,parse_analyze_hook,create_command_tag_hook" being undeclared variables in cypher_analyze.c.I also can't find above three variables in postgres headers. Please help!

Contact Details

No response

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

xiechuanjie avatar Oct 08 '24 02:10 xiechuanjie

Hello,

So this project is undergoing a lot of changes right now and its not in the most usable condition, sorry about that.

This version of PostGraph is using a modified version of Postgres. The thesis being that with a feel small additions to Postgres, you can add a new query language. *EDIT: and storage engine

You can find the modified version of Postgres here. https://github.com/PostGraphDB/postgres_for_postgraph

I also made a branch from the project before I added these new hooks, https://github.com/PostGraphDB/postgraph/tree/old_main. You can take a look while I get the main branch to a workable state

JoshInnis avatar Oct 08 '24 03:10 JoshInnis

Sidenote: the PostGIS integration is not done, if you need specific functions, ask me and I will add them, that part is relatively simple and does not take a large amount of time

JoshInnis avatar Oct 08 '24 03:10 JoshInnis

@JoshInnis Thank you very much for your prompt reply, my suggestion is to add the 3D data type PolyhedralSurface for postgraph , which will help a lot in storing 3D data graph for architecture (such as BIM). Thanks again!

xiechuanjie avatar Oct 08 '24 05:10 xiechuanjie

The base data type is supported. I added a test case to confirm it. The functions are not there yet, I'll try to do those tomorrow. I assume you want the index support too? That will take a little while longer.

JoshInnis avatar Oct 08 '24 05:10 JoshInnis

It was a longer day at work than expected. I supported most of the PostGIS regression cases for PolyhedralSurface, but couldn't do them all.

I'll keep adding more, I need a break from reconciling the PostGraph and Postgres lexar anyways.

JoshInnis avatar Oct 09 '24 06:10 JoshInnis

@JoshInnis Thank you very much for your hard work, if it becomes available I will immediately carry out application testing.

xiechuanjie avatar Oct 09 '24 13:10 xiechuanjie