search

vesoft-inc / nebula-importer

Nebula Graph Importer with Go
Apache License 2.0
90 stars 60 forks source link

Support concat for VID #244

Closed veezhang closed 1 year ago

veezhang commented 1 year ago

Introduction

Support concat with constant, field or mixed for VID.

Contents

Related work

wey-gu commented 1 year ago

If we support this, a lot of Transform Work will be no longer needed(it will be extremely friendly to users!❤️), and we could extend its concat capability to vids and any other columns.

veezhang commented 1 year ago

@wey-gu I feel that the props should be split as small as possible. Is it necessary to support concat for props?

wey-gu commented 1 year ago

@wey-gu I feel that the props should be split as small as possible. Is it necessary to support concat for props?

If from implementation perspective, it's a lot of effort to do so, please ignore it, I am not sure if it's necessary, as the reason I would like to request concat was on VID. But there could be the such need, for instance, just in case, we have First name, Last name cols in the data source and would like to have Name as prop :-D.