sarutak opened a new pull request, #2430:
URL: https://github.com/apache/avro/pull/2430

   ## What is the purpose of the change
   
   According to the specification, each portion of a namespace separated by dot 
should be `[a-z,A-Z,_][a-z,A-Z,0-9_]`.
   https://avro.apache.org/docs/1.11.1/specification/#names
   
   ```
   The name portion of the fullname of named types, record field names, and 
enum symbols must:
   
       start with [A-Za-z_]
       subsequently contain only [A-Za-z0-9_]
   
   A namespace is a dot-separated sequence of such names. The empty string may 
also be used as a namespace to indicate the null namespace. Equality of names 
(including field names and enum symbols) as well as fullnames is case-sensitive.
   
   The null namespace may not be used in a dot-separated sequence of names. So 
the grammar for a namespace is:
   
     <empty> | <name>[(<dot><name>)*]
   ```
   
   But the current Java binding accept invalid namespaces.
   This PR aims to fix this issue.
   
   ## Verifying this change
   Added new test and it passed on my laptop.
   
   ## Documentation
   No new features added.


-- 
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: dev-unsubscr...@avro.apache.org

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

Reply via email to