StewartJingga
StewartJingga
> @StewartJingga Is owner information is must in Compass APIs? Is shouldn't be, right? No, it does not require owner information. But it does require a header (or in grpc...
@rohilsurana @AkarshSatija @ravisuhag @mabdh
@ravisuhag ok so in `Compass.Asset`, it would be like below ``` type Asset struct { ...other fields URN string CreateTime time.Time UpdateTime time.Time CreatedAt time.Time UpdatedAt time.Time } ``` will...
> Yes, this makes it confusing for sure. One way could be to rename compass related timestamp to ingestedAt etc. Or you think first approach is better? The only problem...
LGTM, just a small concern if `ml_feature_table` is a good type name? cc: @sudo-suhas @ravisuhag
Nice @sudo-suhas! Totally forgot that we now can do this
~~Also CMIIW, this will require changes. on meteor's `compass` sink right? @sudo-suhas~~ Edit: Just tested this and it works fine, it seems like we still keep the API contract and...
@bsushmith what would be the expected behaviour when merging slice? Should it always append? If not, how do we check if incoming item in the list already exists in the...
LGTM, but found an issue that may or may not be solved in this PR https://github.com/odpf/compass/issues/182 It is out of scope for sure, but can this be easily solved in...