Sasha, The citing is from https://ask.opendaylight.org/question/2377/odl-coding-style-sun-or-google-java/. Some newer open source code is using google style. As mentioned in
https://www.quora.com/What-is-the-most-universal-Java-code-style-guidelines-and-why, "I would say the Sun Coding Convention is the most widely used as it is the oldest. However, I would consider using more recent coding conventions as this is based on lessons learnt." FYI: OpenDaylight, the largest open source SDN controller. AD_SAL is a flavor of the Service Abstraction Layer created for the Controller project, whose task is primarily to create a layer against which the applications can be developed without assuming the underlying SDN protocol it's talking about. SDN is software defined network. It is related to networking. https://wiki.opendaylight.org/view/OpenDaylight_Controller:AD-SAL (this page is deprecated) Model-Driven SAL (MD-SAL) is a set of infrastructure services aimed at providing common and generic support to application and plugin developers. https://wiki.opendaylight.org/view/OpenDaylight_Controller:MD-SAL On Fri, Jul 7, 2017 at 10:55 AM, Alexander Kolbasov <[email protected]> wrote: > Thanks Lina. What is AD_SAL and MD_SAL? Who were you citing as the person > preferring yo use Google Java style? Is there any description of the > differences between the two? > > - Alex > > On Fri, Jul 7, 2017 at 5:51 PM, Na Li <[email protected]> wrote: > > > Sasha, > > > > found the following online, and I think we can go for google style. > > > > "Sun coding style was used in original contribution of Controller and > > AD-SAL and is mostly used in AD-SAL. > > > > Google Java Style is used in newer code such as YANGTools, MD-SAL, etc... > > so I would say prefer Google Java Style, since community decided on going > > with that style." > > > > I like the idea of enforcing style on new code. I can take a look on how > > easy to do that. > > > > Thanks, > > > > Lina > > > > On Fri, Jul 7, 2017 at 9:50 AM, Alexander Kolbasov <[email protected]> > > wrote: > > > > > Hello, > > > > > > What would people thing about enforcing code style for Apache Sentry > > code? > > > There is a maven/IntelliJ plugin - checkstyle ( > > > http://checkstyle.sourceforge.net/index.html) that can help enforcing > > the > > > style. > > > > > > The problem is that the current code isn't compliant and fails a lot of > > > checks and it wouldn't be reasonable to mass-modify a lot of existing > > code > > > to comply. Instead I would suggest that the new code and the code that > is > > > modified extensively should comply and we can enforce this at code > review > > > time. It might be interesting to research whether checkstyle tool can > be > > > configured with a list of exclude/include files which can be modified > as > > > more files are made compliant. > > > > > > Another interesting issue is related to the style details. There are > two > > > styles supported out of the box - original Sun Java style and Google > > style. > > > I am not sure what is the difference between the, it would be > interesting > > > to research. > > > > > > What do you think about this? > > > > > > - Alex > > > > > >
