search

apache / incubator-graphar

An open source, standard data file format for graph data storage and retrieval.
https://graphar.apache.org/
Apache License 2.0
217 stars 46 forks source link

[format] Unify name from `target` to `destination` #587

Closed Thespica closed 1 month ago

Thespica commented 1 month ago

Describe the enhancement requested

There are naming conflict with single concept in proto: AdjListType call it target image meanwhile EdgeInfo call it destination image

We can unify them to destination.

Component(s)

Format

SemyonSinchenko commented 1 month ago

@Thespica May we use dst and src instead of destination and source?

Thespica commented 1 month ago

@Thespica May we use dst and src instead of destination and source?

I think it will be better to use full name, cause source and destination are more like specific concepts. We should unify usages of them.

(appended)emmmmm... it do bother to use full name everywhere, I consider we can use full name for enum or constant string, use src and dst in higher API?