Redesign scan vertices

[Shylock-Hg]

Please check the FAQ documentation before raising an issue

Describe the bug (required)

Before #3328 we scan vertices by scan tags, but now it's mistake. So we need to redesign scan vertices.

Your Environments (required)

• OS: uname -a
• Compiler: g++ --version or clang++ --version
• CPU: lscpu
• Commit id (e.g. a3ffc7d8)

How To Reproduce(required)

Steps to reproduce the behavior:

1. Step 1
2. Step 2
3. Step 3

Expected behavior

Additional context

[cangfengzhs]

According to the current implementation, tags will be specified in Scan vertex, and only vertexes with these tags will be accessed. If it is traversed according to vertexKey, all points will be traversed. I'm not quite sure if this logic is appropriate.

[Shylock-Hg]

According to the current implementation, tags will be specified in Scan vertex, and only vertexes with these tags will be accessed. If it is traversed according to vertexKey, all points will be traversed. I'm not quite sure if this logic is appropriate.

Yes, current implementation will ignore vertex without any tags.

[cangfengzhs]

Yes, current implementation will ignore vertex without any tags.

So do we have to change it to traverse all the vertices regardless of whether there is a corresponding tag? I'm not sure how much impact this change will have

[Shylock-Hg]

Yes, ScanVertices should return one row for each vertex regardless of whether there are any tags on it.

[xtcyclist]

I think we shouldn't do any redesign for 3.4. Sounds too big. @Shylock-Hg what do you think?

[Shylock-Hg]

I think we shouldn't do any redesign for 3.4. Sounds too big. @Shylock-Hg what do you think?

It's a bug of vertex key, now it's removed. I close it. @cangfengzhs