Re: [PR] Implement multiple label [age]

["rafsun42 (via GitHub)" <gi...@apache.org>]

rafsun42 commented on PR #2082:
URL: https://github.com/apache/age/pull/2082#issuecomment-2342484834

   @MuhammadTahaNaveed Thanks for reporting this. The logic for generating intersection relation was shortsighted. It appends the individual label names together without any separator. So, both `CREATE (:A:B:C)` and `CREATE (:AB:C)` generates the same relation name `_agr_ABC`. The expectation is they should generate different relation name.
   
   One solution can be using a special separator character and disallowing it in label names. For example, the `%` character  (it is already disallowed I believe). So, the intersection relations will be `_agr_A%B%C` and `_agr_AB%C` respectively.
   
   What's your thoughts?
   
    


-- 
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: commits-unsubscribe@age.apache.org

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