"Higuchi, Daisuke" <higuchi.dais...@jp.fujitsu.com> writes: > By adding flex option '-i', replication command parser could be more > flexible. > This option is already used for syncrep_scanner.c, so it is not strange to > add for repl_scanner.c too.
Really? That wasn't an especially bright idea IMO, because it means that the scanner's behavior will be dependent on the locale flex was run in. An example here is that Turkish locale is likely to have a different idea of what "FIRST" matches than other locales do. Indeed, if I run the flex build in a non-C locale, I get warnings like $ LANG=en_US /usr/bin/flex -b -CF -p -i -o'syncrep_scanner.c' syncrep_scanner.l syncrep_scanner.l:91: warning, the character range [\200-\377] is ambiguous in a case-insensitive scanner syncrep_scanner.l:91: warning, the character range [\200-\377] is ambiguous in a case-insensitive scanner We'd probably be better off to implement case-insensitivity the hard way. There is a reason why none of our other flex scanners use this switch. While I'm whining ... it looks like the other flex options selected here were cargo-culted in rather than being thought about. Surely we don't run syncrep_scanner often enough, nor over so much data, that it's a good tradeoff to use the options for larger tables. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers