CapnSpek commented on issue #870:
URL: https://github.com/apache/age/issues/870#issuecomment-1528167892

   The bug can be easily fixed by modifying the
   "static int get_type_sort_priority(enum agtype_value_type type)"
   function in agtype_util.c file.
   The reason for the bug is that there is no priority assigned for neither 
edge, nor path.
   The function returns -1 for both.
   
   And because the value is checked like this
   
   ```
   res = (get_type_sort_priority(va.type) <
                      get_type_sort_priority(vb.type)) ?
                         -1 :
                         1;
   ```
   
   It means even if the function returns the same value for both the types, we 
get 1 (true) result for both
   edge > path
   and
   path > edge.
   
   I would suggest modifying the function to declare priorities concretely.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to