Created an documentation an #2935 <https://github.com/apache/incubator-dubbo/issues/2935> issue (Creating dubbo dubbo-rpc-api java documentation) and will be on it, could someone tag or assigned it to me.
Regards Imteyaz On Sat, Dec 8, 2018 at 7:19 PM Huxing Zhang <[email protected]> wrote: > Hi Imteyaz, > > On Fri, Dec 7, 2018 at 9:30 PM Imteyaz Khan <[email protected]> > wrote: > > > > Hi All, > > I have created the pull > > <https://github.com/apache/incubator-dubbo/pull/2921>request for github > > issue 2884 (which is getting used in the email thread). This PR is for > > dubbo-filter modules (dubbo-filter-cache and dubbo-filter-validation). > > I've reviewed your pull request and it looks good to me, I have merged > your pr already. > I think it is really a good starting point, hope to see your further > contributions! > > > While this is in PR is in review state I will take the other module of > > writing javadoc and comments, I was thinking to start another email > thread > > for another documentation task or should I continue the same. > > Both are ok, it is up to you to decide. :) > > > > > https://github.com/apache/incubator-dubbo/pull/2921 > > > > > > Regards > > Imteyaz > > > > On Thu, Dec 6, 2018 at 4:57 PM Ian Luo <[email protected]> wrote: > > > > > :) welcome to the community. > > > > > > On Thu, Dec 6, 2018 at 2:58 PM Imteyaz Khan <[email protected]> > > > wrote: > > > > > > > Ian, > > > > > > > > Thanks for guiding me. I am on it. > > > > > > > > Regards > > > > Imteyaz > > > > > > > > > > > > On 6 Dec 2018 11:49 a.m., "Ian Luo" <[email protected]> wrote: > > > > > > > > Hi Khan, > > > > > > > > Feel free to start from any module to enhance the existing JavaDoc > and > > > > comments, and submit the pull request when you feel it's ready. I am > so > > > > glad that there's native speaker willing to join this effort :) > > > > > > > > Thanks, > > > > -Ian. > > > > > > > > > > > > On Thu, Dec 6, 2018 at 4:14 AM Imteyaz Khan <[email protected]> > > > > wrote: > > > > > > > > > I have just started documenting module 'dubbo-filter', below are > > > example > > > > > of mine, this is just to make sure I am in align with the > documentation > > > > > style > > > > > > > > > > /** > > > > > * CacheFilter is a core component of dubbo.Enabling <b>cache</b> > key > > > > > of service,method,consumer or provider dubbo will cache method > return > > > > > value. > > > > > * Along with cache key we need to configure cache type. Dubbo > default > > > > > implemented cache types are > > > > > * <li>lur</li> > > > > > * <li>threadlocal</li> > > > > > * <li>jcache</li> > > > > > * <li>expiring</li> > > > > > * > > > > > * <pre> > > > > > * e.g. 1)<dubbo:service cache="lru" /> > > > > > * 2)<dubbo:service /> <dubbo:method name="method2" > > > > > cache="threadlocal" /> <dubbo:service/> > > > > > * 3)<dubbo:provider cache="expiring" /> > > > > > * 4)<dubbo:consumer cache="jcache" /> > > > > > * > > > > > *If cache type is defined in method level then method level type > will > > > > > get precedence. According to above provided > > > > > *example, if service has two method, method1 and method2, method2 > > > > > will have cache type as <b>threadlocal</b> where others will > > > > > *be backed by <b>lru</b> > > > > > *</pre> > > > > > * > > > > > * @see org.apache.dubbo.rpc.Filter > > > > > * @see org.apache.dubbo.cache.support.lru.LruCacheFactory > > > > > * @see org.apache.dubbo.cache.support.lru.LruCache > > > > > * @see org.apache.dubbo.cache.support.jcache.JCacheFactory > > > > > * @see org.apache.dubbo.cache.support.jcache.JCache > > > > > * @see > > > > org.apache.dubbo.cache.support.threadlocal.ThreadLocalCacheFactory > > > > > * @see org.apache.dubbo.cache.support.threadlocal.ThreadLocalCache > > > > > * @see > org.apache.dubbo.cache.support.expiring.ExpiringCacheFactory > > > > > * @see org.apache.dubbo.cache.support.expiring.ExpiringCache > > > > > * > > > > > */ > > > > > @Activate(group = {Constants.CONSUMER, Constants.PROVIDER}, value = > > > > > Constants.CACHE_KEY) > > > > > public class CacheFilter implements Filter { > > > > > > > > > > .... > > > > > > > > > > ..... > > > > > > > > > > } > > > > > > > > > > > > > > > On Wed, Dec 5, 2018 at 8:20 PM Imteyaz Khan < > [email protected]> > > > > > wrote: > > > > > > > > > > > Just to make sure I have not misunderstood what mentioned > earlier by > > > > > > 'Filter' is 'dubbo-filter' correct? > > > > > > > > > > > > On Wed, Dec 5, 2018 at 7:00 PM Imteyaz Khan < > [email protected]> > > > > > > wrote: > > > > > > > > > > > >> Thanks a lot for the start, much appreciated. > > > > > >> > > > > > >> > > > > > >> > > > > > >> On Wed, Dec 5, 2018 at 6:31 PM yuhang xiu <[email protected]> > > > wrote: > > > > > >> > > > > > >>> In addition, we strongly encourage open discussion. > > > > > >>> You can post your comments on a Filter directly here. There > will be > > > > > some > > > > > >>> people who know more about dubbo to review and discuss. > > > > > >>> > > > > > >>> Of course, any problem can be raised here. We will answer as > soon > > > as > > > > > >>> possible. > > > > > >>> > > > > > >>> yuhang xiu <[email protected]> 于2018年12月5日周三 下午8:58写道： > > > > > >>> > > > > > >>> > very sorry. I have some work now, so I didn't respond to your > > > email > > > > > in > > > > > >>> > time. > > > > > >>> > I am very willing to help you with the completion of the > > > comments. > > > > If > > > > > >>> you > > > > > >>> > are a newbie, I suggest you start with a simple logic. > > > > > >>> > Which parts of the logic are relatively simple? At present, I > > > think > > > > > >>> that > > > > > >>> > our filter-related implementation is relatively simple (such > as > > > > > >>> > ActiveLimitFilter). You don't know the principle of dubbo at > all, > > > > and > > > > > >>> you > > > > > >>> > can understand the code and complete the comments more > easily. > > > You > > > > > can > > > > > >>> > start with the Filter related class and complete some of the > > > > > comments. > > > > > >>> > > > > > > >>> > When you think that you know enough about dubbo, you can do > some > > > > core > > > > > >>> > module comments, such as protocol-related comments. > > > > > >>> > > > > > > >>> > Imteyaz Khan <[email protected]> 于2018年12月5日周三 > 下午3:05写道： > > > > > >>> > > > > > > >>> >> Hi All, > > > > > >>> >> Trust you all are doing well. I am new to this community > or > > > > even > > > > > >>> new to > > > > > >>> >> any open source development community. Had a chance to go > > > through > > > > > one > > > > > >>> of > > > > > >>> >> the dubbo github issue #2884('We need more code comments'), > > > which > > > > > was > > > > > >>> >> marked as 'good first issue'. So thought it could be a good > > > start > > > > > for > > > > > >>> me > > > > > >>> >> to > > > > > >>> >> start from here, but being new to open source and also to > dubbo > > > I > > > > am > > > > > >>> very > > > > > >>> >> nervous from where and how to start. I will be great if > team can > > > > > help > > > > > >>> me > > > > > >>> >> and guide me too to assist you in any way I can. > > > > > >>> >> > > > > > >>> >> Regards > > > > > >>> >> Imteyaz > > > > > >>> >> > > > > > >>> > > > > > > >>> > > > > > >> > > > > > > > > > > > > > > > > -- > Best Regards！ > Huxing >
