Re: Keyword table constness in jsonpath scanner.

2019-03-17 Thread Alexander Korotkov
On Sat, Mar 16, 2019 at 10:47 PM Mark G wrote: > While looking over the new jsonpath stuff I noticed the keyword table > wasn't declared const. Shouldn't the table and the actual keyword > strings both be declared const? Perhaps something like the attached > (untested) patch. Looks good to me. P

Keyword table constness in jsonpath scanner.

2019-03-16 Thread Mark G
Hello all, While looking over the new jsonpath stuff I noticed the keyword table wasn't declared const. Shouldn't the table and the actual keyword strings both be declared const? Perhaps something like the attached (untested) patch. -Mark diff --git a/src/backend/utils/adt/jsonpath_scan.l b/src/b