fdb-document-layer icon indicating copy to clipboard operation
fdb-document-layer copied to clipboard

Resolves #17: Update explain() output with user readable keys

Open drkannan opened this issue 5 years ago • 10 comments

This PR resolves #17

Explain() output will be much more useful if the keys are user readable, instead of FDB keys. Used the existing decode_key_part function to achieve this.

drkannan avatar Aug 26 '19 08:08 drkannan

Can one of the admins verify this patch?

fdb-build avatar Aug 26 '19 08:08 fdb-build

Can one of the admins verify this patch?

fdb-build avatar Aug 26 '19 08:08 fdb-build

Can one of the admins verify this patch?

fdb-build avatar Aug 26 '19 08:08 fdb-build

Can one of the admins verify this patch?

fdb-build avatar Aug 26 '19 08:08 fdb-build

@apkar could you please verify this patch

senthil-db-expert avatar Aug 29 '19 01:08 senthil-db-expert

@fdb-build test this please

apkar avatar Aug 29 '19 17:08 apkar

@fdb-build test this please

AlvinMooreSr avatar Aug 30 '19 17:08 AlvinMooreSr

@fdb-build test this please

apkar avatar Sep 06 '19 17:09 apkar

@senthil-db-expert Can you give some explanation on the usage of DVTypeCode::SPL_CHAR please? Also can you add some tests please? test_planner.py is probably the best place.

apkar avatar Sep 06 '19 17:09 apkar

@apkar Added explanations and test cases.

senthil-db-expert avatar Sep 12 '19 01:09 senthil-db-expert