You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@age.apache.org by GitBox <gi...@apache.org> on 2021/06/25 05:08:35 UTC

[GitHub] [incubator-age] JoshInnis commented on pull request #87: Adding AGType parsers and driver supports for Python & Go

JoshInnis commented on pull request #87:
URL: https://github.com/apache/incubator-age/pull/87#issuecomment-868207798


   There is a data type that Agtype supports called numeric. Agtype values that are numeric end in '::numeric'. It looks like both python and go parsers are not covering this case. I would also recommend taking a look at https://github.com/apache/incubator-age/blob/master/drivers/Agtype.g4. The type annotation system '::vertex', '::numeric' has the potential to expand in the future if we support more datatypes. Accounting for that in the parser now may save some headaches in the future.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org