Github user LosD commented on the issue:

    https://github.com/apache/metamodel/pull/163
  
    Took a look (phone code-browsing was not as bad as I thought it would be) . 
Code looks fine, but the whole idea is pretty terrible. We have plenty of 
problems with datastores being vulnerable to order change (just look at our 
integration tests where half the issues are due to order being assumed), and 
this will make it worse.
    
    As I read from the issue, its MEANT for single table datastores, but that's 
not the implementation. This _will_ be used for more and it_will_ break when 
they update their databases.
    
    I'd either drop it completely or find a format that can only be used for 
single-table datastores, not even for multi table datastores that currently 
contains a single table. 


---

Reply via email to