yangxk1 opened a new issue, #814:
URL: https://github.com/apache/incubator-graphar/issues/814

   ### Describe the enhancement requested
   
   In the java-info implementation, when parsing AdjacentList or PropertyGroup, 
if the prefix field is missing, it is currently treated as null/empty, which 
can lead to issues later. Referring to the C++ logic, if prefix is empty, it is 
set to a reasonable default value (e.g., PropertyGroup uses concatenated 
property names, AdjacentList uses type name as the directory name. See the C++ 
reference implementation: 
[graph_info.cc#L109](https://github.com/apache/incubator-graphar/blob/192a1dc4b8e4ce9daf6175e32520a6001942b492/cpp/src/graphar/graph_info.cc#L109)).
 I have completed a simple demo for this logic 
(https://github.com/yangxk1/incubator-graphar/commit/c84c162c3d8a8096dbd303d67fe7612a39b3424f),
 but would like confirmation whether there are other locations in the codebase 
that should also add this kind of default value handling. Please review all 
AdjacentList and PropertyGroup related parsing/construction code to ensure 
consistent default prefix logic and to avoid missing cases. 
   
   
   ### Component(s)
   
   Java


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to