catarinaaa opened a new pull request, #13502:
URL: https://github.com/apache/iceberg/pull/13502

   ## Description
   On AWS Glue it's possible to create namespaces with a dot character however 
in iceberg this is not considered a valid namespace. Therefore this PR adds the 
. "dot" as a valid character on the DB pattern.
   
   This is done by changing the `GLUE_DB_PATTERN` variable in 
`IcebergToGlueConverter`. Also updates the error message.
   
   ## Testing
   Added more success cases to positive test on `testToDatabaseName`.
   
   ## Questions
   Do you think this can cause bad interoperability with the existing code, 
since we are also splitting namespaces with a dot character? 
   Any reason why not do this?


-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to