Github user QiangCai commented on the issue:

    https://github.com/apache/incubator-carbondata/pull/508
  
    @ravipesala 
    In my opinion, If some profile is active by default and only contains some 
property elements, we can remove this profile and add these property elements 
into properties element.
    For assembly/pom.xml, I prefer to add these three provided property 
elements into properties element, but not each profile.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to