novatechflow commented on PR #839:
URL: https://github.com/apache/wayang/pull/839#issuecomment-6335688098

   Two small things, reproduced on Java 17:
   - Mixed quoting breaks: `"PUBLIC".target_append` becomes 
`"""PUBLIC"""."TARGET_APPEND"`, and causes lookup failures.
   - Case folding depends on system locale: under Turkish locale, input_table 
becomes `"İNPUT_TABLE"` and fails. Use `Locale.ROOT`, that should avoid that.
   
   I think the quoting utility needs per-segment quote handling and 
locale-independent folding before approval. Thoughts?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to