Acesslog dateformat enhancemnet #3274

khanimteyaz · 2019-01-18T19:48:12Z

What is the purpose of the change

AcceLogFilter message creation and last and current check was performing by each time simple date formate object creation. In this version of code I have use the ThreadLocal for each thread wise to create simple date format object instead of each call

#3026

Previous PR I have closed as it was too old and due to recent major StringUtils changes 500+ files were showing as modified files and around 8000+ line of code changes, which will may create some confusion. To avoid this confusion have raised new PR.

In previous PR there was request change was asked by @zonghaishang , which (not thread safe)I beleive I have adrressed.

In this PR

I have moved the access log creation and writing to file into a single thread and instead of each time creating a SimpleDateFormat using FastDateFormat formatter with new created DateUtil.
Refactored code to separate our and group related task in separate method and have enhance the readability by using
- Method renaming
- Reducing big methods to small.
- DateUtil utility using FastDateFormat

This PR is a resurrection of old PR

3027 (Simple date format each time new object creation removed #3027) PR
3080 (Simple date format each time new object creation removed #3080)
3.Acesslog dateformat enhancemnet #3090
Sorry for the inconvenience

Brief changelog

XXXXX

Verifying this change

Runnig UT and verifying generated access log.

Follow this checklist to help us incorporate your contribution quickly and easily:

Make sure there is a [GITHUB_issue] (AccessLogFilter simple date format reduce instance creation #3026) field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
If this contribution is large, please follow the Software Donation Guide.

Update from upstream

master synch

Sync of master

Master sync

master sync

codecov-io · 2019-01-18T20:13:17Z

Codecov Report

Merging #3274 into master will increase coverage by 0.11%.
The diff coverage is 85.82%.

@@             Coverage Diff              @@
##             master    #3274      +/-   ##
============================================
+ Coverage     63.84%   63.95%   +0.11%     
  Complexity       71       71              
============================================
  Files           661      662       +1     
  Lines         28514    28551      +37     
  Branches       4793     4789       -4     
============================================
+ Hits          18205    18261      +56     
+ Misses         8095     8086       -9     
+ Partials       2214     2204      -10

Impacted Files	Coverage Δ	Complexity Δ
...a/org/apache/dubbo/rpc/filter/AccessLogFilter.java	`83.33% <80.95%> (+8.88%)`	`0 <0> (ø)`	⬇️
...va/org/apache/dubbo/rpc/support/AccessLogData.java	`90.14% <90.14%> (ø)`	`0 <0> (?)`
.../remoting/transport/netty4/NettyServerHandler.java	`61.9% <0%> (-9.53%)`	`0% <0%> (ø)`
.../remoting/transport/netty4/NettyClientHandler.java	`61.4% <0%> (-7.02%)`	`0% <0%> (ø)`
...ng/transport/dispatcher/all/AllChannelHandler.java	`51.42% <0%> (-5.72%)`	`0% <0%> (ø)`
.../org/apache/dubbo/remoting/ExecutionException.java	`15.78% <0%> (-5.27%)`	`0% <0%> (ø)`
...org/apache/dubbo/rpc/protocol/AbstractInvoker.java	`64.51% <0%> (-3.23%)`	`0% <0%> (ø)`
...exchange/support/header/HeaderExchangeHandler.java	`55.81% <0%> (-2.33%)`	`0% <0%> (ø)`
...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java	`87.85% <0%> (-1.87%)`	`0% <0%> (ø)`
...rg/apache/dubbo/common/timer/HashedWheelTimer.java	`57.83% <0%> (-0.35%)`	`0% <0%> (ø)`
... and 14 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 713a3ea...4f6d008. Read the comment docs.

dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/filter/AccessLogFilter.java

dubbo-common/src/main/java/org/apache/dubbo/common/utils/DateUtil.java

dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/filter/AccessLogFilter.java

dubbo-common/pom.xml

… package modified version

…khanimteyaz/incubator-dubbo into acesslog-dateformat-enhancemnet

khanimteyaz · 2019-01-22T07:08:52Z

@carryxyh @beiwei30 I have made the changes, could you plz have a look.

carryxyh · 2019-01-22T07:48:27Z

Hi, Seems like the source code of FastDateFormat is very heavy...

There are many places in the source code that currently use SimpleDateFormat. If we introduce the source code, we should replace all the SimpleDateFormat and share the DateFormat.

@khanimteyaz
Would you like to make an issue and optimize them? :)

beiwei30 · 2019-01-22T08:17:45Z

Hi, Seems like the source code of FastDateFormat is very heavy...

There are many places in the source code that currently use SimpleDateFormat. If we introduce the source code, we should replace all the SimpleDateFormat and share the DateFormat.

@khanimteyaz
Would you like to make an issue and optimize them? :)

Agree with @carryxyh, @khanimteyaz we should not rely on FastDateFormat. Would you mind to change it to rely on JDK's DateFormat?

khanimteyaz · 2019-01-22T13:23:00Z

Hi, Seems like the source code of FastDateFormat is very heavy...
There are many places in the source code that currently use SimpleDateFormat. If we introduce the source code, we should replace all the SimpleDateFormat and share the DateFormat.
@khanimteyaz
Would you like to make an issue and optimize them? :)

Agree with @carryxyh, @khanimteyaz we should not rely on FastDateFormat. Would you mind to change it to rely on JDK's DateFormat?

@beiwei30 , you want me to revert the change and use DateFormat or we should us FastDateFormat and replace all existing SimpleDateFormat to use FastDateFormat using DateUtil? Sorry for being confuse here 😃

khanimteyaz · 2019-01-27T18:04:14Z

@beiwei30 could you provide our thought on this?

tswstarplanet · 2019-01-28T00:43:51Z

We can consider using new api in java.time.* packages to replace old java time apis.
The new apis provider immutable classes about time, so they are thread safe.

kun-song · 2019-01-28T01:52:58Z

We can consider using new api in java.time.* packages to replace old java time apis.
The new apis provider immutable classes about time, so they are thread safe.

+1, Yes, the new time API is much better.

beiwei30 · 2019-01-31T08:48:52Z

Hi, Seems like the source code of FastDateFormat is very heavy...
There are many places in the source code that currently use SimpleDateFormat. If we introduce the source code, we should replace all the SimpleDateFormat and share the DateFormat.
@khanimteyaz
Would you like to make an issue and optimize them? :)

Agree with @carryxyh, @khanimteyaz we should not rely on FastDateFormat. Would you mind to change it to rely on JDK's DateFormat?

@beiwei30 , you want me to revert the change and use DateFormat or we should us FastDateFormat and replace all existing SimpleDateFormat to use FastDateFormat using DateUtil? Sorry for being confuse here 😃

correct.

…khanimteyaz/incubator-dubbo into khanimteyaz-acesslog-dateformat-enhancemnet

beiwei30 · 2019-01-31T09:11:32Z

@khanimteyaz I submitted a pull request khanimteyaz#11 in your repository. Would you mind to take a look?

My basic idea is: since we are now formatting date and time in single thread, it is safe to use a singleton formatter without any risk.

…enhancemnet switch to use jdk's dateformatter

khanimteyaz · 2019-02-01T07:34:05Z

@beiwei30 I have merged ur PR in branch to this branch. Could you please look into this. Thanks for your support.

khanimteyaz · 2019-02-01T09:37:41Z

@satansk could you review, as you requested for changes.

beiwei30 · 2019-02-02T02:30:00Z

LGTM now.

khanimteyaz · 2019-02-02T04:22:02Z

Thanks @beiwei30

@OverRide

* Release resource after use in ConfigParserTest (#3127) Release resource after use in ConfigParserTest * add javadoc for registry and some code (#3140) add javadoc for registry and optimize code * Correct spelling error (#3146) * Code optimization (#3118) * code optimization * useless import * optimization * Code rule (#3016) * code optimization (#3167) code optimization * Add javadoc for dubbo-serialization module(#3002). (#3004) Add javadoc for dubbo-serialization module(#3002). * optimize ReconnectTimerTask's log output (#3162) * optimize log output * Separate logs for reconnect and close * remove reconnect exception log * add some small optimize (#3171) * modify some log describe * use java8 lambda expression * fix telnet trace times is always 1 (#3038) * fix telnet trace times is always 1 * use StringUtils determine if the string is empty * Fix 3105 , make invoke command with Json string parameter without "class" key * Fix 3105 ，Keep the class key to support overloaded methods * optimize InvokerTelnetHandlerTest * Upgrade junt to junit5 (#3149) * upgrade junit to junit5 * modify test * 批量修改upgrade_junt_to_junit5 * 删除多余的文件 * fi test case * Disabled soem test case temporarily * upgrade junit to junit5 and batch modify test case * copy some code from jupiter5.4.0.M1 for some issues * 修改rat福泽 * update rat path * revert case * add junit-platform-surefire-provider to maven-surefire-plugin * update dependency * fix coverage issue (#1) * use jupiter 5.4.0-M1 and remove junit5 source code * Format change. * fix wrong word spelling (#3217) * Fix provicer --> provider (#3222) * Optimize the code: use logger instead of printStackTrace(). (#3202) * Optimize code: remove unnecessary judgment code. (#3196) * Optimize the code: fix CallbackServiceCodec.java exportOrunexportCallbackService method issue. (#3199) * Optimize the code: fix url to null, NullPointerException, change private variable to camel mode. * Optimize the code: exportOrUnexportCallbackService method camel mode. * Optimize the code: fix method:encodeInvocationArgument private callbackStatus is camel writing. * Optimize the code: fix name issue * Exporter is a noun, we should use a verb here, like Export. * The generics that can be inferred automatically are also deleted. * Refactor telnet invoke command (#3210) * refactor telnet invoke command * add select command for telnet * fix test case * Remove deprecated AnnotationBean, please refer #1485 for the new method to replace. (#3232) * [Dubbo-3231]keep TagRouter consistent with 2.6.x (#3233) * keep TagRouter consistent with 2.6.x * refactor filterUsingStaticTag using lambda in tagRouter * Modified to lower camel case (#3003) * wrong event setting (#3043) * wrong event setting * modify event seeting * modify * call the util method (#3230) * Code optimization, call the util method * mofidy * modify * * import package * Qos heart (#3170) * qos heart question fix #3165 * modify * judge if it's a IdleStateEvent * add UT * modify * Merge pull request #3246 from cvictory:2.7.0-release remove gson from dubbo.jar in shading mode , and change to dependency way. * just for modify comments and imports * remove gson from dubbo.jar in shading mode, add dependency * Extract compareTo impl to Router interface and concrete Router only responsible for provide priority. (#3240) something is waiting for us to disscuss: 1. Every Route implement should set a priority? 2.#3249 * Protocol compatibility (#3254) add default method into Protocol . * remove getContext() (#3235) * modify some typos (#3257) * modify some typos * fix some other addionalParameterKeys and paramter typos * Merge 2.6.6 source code into 2.7 (#3241) * Merge 2.6.6 source code into 2.7 * Fixed logging level for #3241 (comment) * Change the implementation for #3241 * Remove the implementation Ordered, because it does not work in Spring Framework : #3241 * Remove the implementation Ordered, because it does not work in Spring Framework : #3241 * Only Optimize Imports * Add activation for the release profile * Update the Java SE link to Java 8. * Move the doclint configuration to correct position. * Deactivate maven release profile. * Fix final name. * Optimize the Apollo extension of Config-Center SPI * must shutdown thread pool when no in use (#3255) * must shutdown thread pool when no in use * Update 2.7.0 release notes * Igonre .patch file. * [maven-release-plugin] prepare release dubbo-2.7.0 * [maven-release-plugin] prepare for next development iteration * [maven-release-plugin] prepare release dubbo-2.7.0 * [maven-release-plugin] prepare for next development iteration * Bring back redis auth UT (#3278) * Optimize code: Fix Constructor to determine illegal logic problems (#3197) * fix return type (#3284) * use standardcharset.utf-8 instead of literal (#3285) * use standardcharset.utf-8 instead of literal * remove unused import * Move the iteration of methods of a service config to the if block of the class have method declared not by Object; remove some useless parameter type (#3282) * Add shutdown command for telnet (#3280) * telnet add shutdown command * refactor rename shutDown to shutdown * remove unregister in doDestroy * unregister the ShutdownHook when the shutdown command invoked * Ignore mvn wrapper binary files. * [maven-release-plugin] prepare release dubbo-2.7.0 * [maven-release-plugin] prepare for next development iteration * Fix typo (#3293) * Improve/heartbeat (#3276) * add the notice of code style * modify the pic * del teh faq.md, move to dubbo admin * improve:remove the heartbeat on server side * improve:change the scope of timer to static * code optimization (#3297) * further enhancement for pull request #3297, also fix an issue introduced in this pull request (#3303) * further enhancement for pull request #3297, also fix an issue introduced in this pull request * rename the variable * enhance the readability * Fix UT conflicts of merging 2.7.0-release * Remove usage of classes in Junit 4 * Remove usage of classes in Junit 4 * Remove usage of classes in Junit 4 * Remove usage of classes in Junit 4 * Remove usage of classes in Junit 4 * remove not used import (#3309) * remove unused import (#3311) * remove not used import * remove unused import * Degrade some UTs in dubbo-config-spring to use junit 4. * use beforeEach and afterEach * Merge pull request #3295, unregister consumer url when client destroyed (referenceconfig#destroy). * fix client reconnect offline provider. * refactor cancel future. * fix client reconnect offline provider. * refactor cancel future. * fix client reconnect offline provider. * refactor cancel future. * fix unregister when client destroyed * Optimize heartbeat (#3299) * Optimize heartbeat. We should cancel the timeout when the client or server is close. * change the hashedWheelTimer's ticks * Optimize tasks keeper. * fix timeout cancel to task cancel. keep task directly. * simply telnet command enabled check logic (#3316) * simply telnet command enabled check * Add comments, manually merge #3181. * Fix for loop reference test pass on protostuff (#3252) * Fix maven compile warning (#3322) * Merge pull request #3174, make timeout filter not work in async way. * Merge pull request #3323, fix double-checked locking. * Merge pull request #2959, fix a bug of service config. * Fix random ut falling in DubboMonitorTest (#3327) * Merge pull request #3017, fixes #2981, refresh invocation's attachments in each invoke. Fixes #2981. * Merge pull request #3141, optimize outbound event and some code formatting. * Merge pull request #3333, add @OverRide for sub-class method. * Fix thrift protocol, use path to locate exporter. (#3331) * Fix thrift protocol, use path to locate exporter. * Fix UT * fix #2842. remove duplicate SPI definitions for 2.7.x (#3340) remove duplicate SPI definitions for 2.7.x * fix org.apache.dubbo replace com.alibaba.dubbo (#3338) org.apache.dubbo replace com.alibaba.dubbo * fix the typo of notification mail list address (#3335) fix the typo of notification mail list address * Review code of TypeDefinitionBuilder (#3064) * Review code of TypeDefinitionBuilder 1. use init method to init builds' list * use single list for all builders. Seems like the builder is thread-safe, we can keep them static and final. * clean code. * [Dubbo-3339] Remove futility check code (#3346) * Remove futility check code * add no-argument constructor method to URL.java fix #3342 (#3350) * Fix NullPointerException when Hessian instantiate URL with JavaDeserializer * Enable ZookeeperMetadataReportTest (#3360) * Package name error (#3354) * some optimize on ExtensionLoader (#3307) * some optimize on ExtensionLoader * make ci rerun * fix compile error * fix ci failure * Fix some unit test failure (#3337) * Fix compilation error fix #3365 (#3366) * [Dubbo-900] Fix 通过 override 修改 hessian协议的提供者的配置不生效 #900 (#3363) * reExport fail fix#900 * modify * use Objects.equals * compare URL for all proxy protocol * delete useless judgment (#3326) * Fix self assignment (#3301) * Fix self assignment * Replace set with list to insure item order * Minor tweak of codestyle * [Dubbo-2423] Multicast demo fails with message "Can't assign requested address". (#3317) * Fix #2423, Multicast demo fails with message "Can't assign requested address" * temporarily disable ipv6 test * simplify map empty judgment (#3376) * 应该是非空才循环，不然会导致在使用redis注册中心时消费者引用不到服务 (#3291) * enhance unit test and logging (#3374) * enhance unit test and logging * enhance logging message * fix unit test * make code clean * follow up for #3291 (#3378) * correct typos,eg: occured -> occurred (#3380) LGTM * follow up for #3376 (#3377) * [Dubbo-3347] Update package name in README file fix#3347 (#3362) * [Enhancement] Replace explicit resource management with try-with-resource (#3281) * first pull request (#3396) * [Dubbo-3361] Make DubboAppender extends from FileAppender (#3383) * Modify deprecated class to updated class in some comments (#3402) * Merge pull request #3341, start to use IdleStateHandler in Netty4. * Enhancement/logger factory (#3389) * polishing LoggerFactory * polishing code using map.computeIfAbsent * fix ci failure * remove unnecessary break in switch * call overloaded method * update as requested * add unit test * move construction of ConfigChangeEvent outside the lambda expression (#3398) * make ConfigChangeEvent immutable (#3403) * Fix license issues (#3382) * Fix license issues * revert change to Netty's copyright * [Dubbo-936]fix The nc command is unstable in the dubbo startup script #936 (#3375) * fix dubbo启动脚本中nc命令不稳定 #936 * modify * refactor ScriptRouter: (#3404) 1, remove priority field, the same as super class 2, remove getUrl method, the same as super class 3, refactor constructor, extract method: getRule, getEngine 4, refactor route, extract method: createBindings, getRoutedInvokers * Clean pom.xml file #3186 (#3211) * update as requested * add meta space size arguments * [Enhancement] Use ThreadLocalRandom and try-with-resource (#3239) * polish * fix code reviews * empty * polish pom.xml (remove test profile and jvm permSize args) (#3407) * update dubbo samples' link (#3413) * Acesslog dateformat enhancemnet (#3274) * #3026 Access log related changed * Reviwe comment incorporated given by @satansk and removed unuded method * Incorporated @beiwei30 review comment, incorporated common-lang3 time package modified version * Added rat entry for common lang3 FastDateFormat related java files * switch back to jdk's dateformatter * refactor loadClass method (#3410) * refactor: expression is always true, remove it (names != null && names.length > 0) * Update dubbo-common/src/main/java/org/apache/dubbo/common/extension/ExtensionLoader.java Co-Authored-By: wanghbxxxx <wanghbxxxx@gmail.com> * Update ExtensionLoader.java * Update ExtensionLoader.java * Update ExtensionLoader.java * Update ExtensionLoader.java * implement pull request #3412 on master branch (#3418) * event of response fix#3244 (#3247) * [Dubbo-3069]Use regular expressions to judge fix #3069 (#3093) * Use regular expressions to judge fix #3069 * moved into Constants class * modify * Unused import * modify * can not put it in front * catch NumberFormatException and return 'null' if necessary * remove recursive call * support .1 and 1. * modify * Support multiple shared links (#2457) * make dubbo support multiple shared links, upgrading RPC throughput * Fix compilation error * Fix compilation error * opti import * if add {} * checkstyle fail * fix getSharedClient referenceCount calculation error bug * 优化 import * Fix the problem that the getSharedClient thread is not safe * Fix the problem that the getSharedClient thread is not safe * Try fixing ci error, https://travis-ci.org/apache/incubator-dubbo/jobs/453185295 * 将DEFAULT_CONNECTIONS_KEY修改成SERVICE_CONNECTIONS_KEY * dubbo.xsd add shareconnections attribute, * Optimize code format * Fix mult connect ghost connect problem * format code * Remove the concept of ghostClientMap and ghost connection. In fact, ghostClient is LazyConnectExchangeClient. At present, the LazyConnectExchangeClient object is added directly in ReferenceCountExchangeClient to realize the mapping relationship with ReferenceCountExchangeClient. The relationship between previous ghostClient and url mapping is not applicable to the current new share. Multiple connections. * Optimize the ReferenceCountExchangeClient and remove the reference to the lazyConnectExchangeClient because it doesn't make much sense; add locks in the close operation of the AbstractClient, because connect, disconnect, and close should not be done at the same time. * format code * try remove close lock * Restore close method * Restore ReferenceCountExchangeClient reference to LazyConnectExchangeClient object * Optimize the logic of using the LazyConnectExchangeClient inside the ReferenceCountExchangeClient; Supplemental shared multi-connected unit test * remove useless catch exception (#3421) * remove duplicated import (#3440) * Update junit to 5.4.0 release version (#3441) * update junit to 5.4.0 release version * remove uesles config * remove duplicated unused method and move unit test (#3446) * Add checkstyle rule for redundant import (#3444) * add checkstyle for redundant import and fix all issue in repo * fix git diff issue * Enhance the java doc of dubbo-container module (#3437) Fixes #2994 * refactor adaptive extension class code creation: extract class AdaptiveClassCodeGenerator (#3419) * refactor adaptive extension class code creation: extract createAdaptiveExtensionClassCode to class AdaptiveClassCodeGenerator * add some comment * add license and comment * remove main method * refactor javassist compiler: extract class CtClassBuilder (#3424) * refactor JavassistCompiler * rename variable names * reformat code * refactor: prepend modifier of constructor, field and method outside the JavassistClassInfo * add null for ClassUtils.getSimpleClassName * rename JavassistClassInfo to CtClassBuilder * fix #2619: is there a problem in NettyBackedChannelBuffer.setBytes(...)? (#3448) * Bugfix/timeout queue full (#3451) replace ArrayBlockingQueue with LinkedBlockingQueue and remove capacity. * extract 2 methods: (#3453) isSetter: test if a method is a setter getSetterProperty: get property for setter, for instance setVersion return "version" * Add delay export test case (#3447) * [Dubbo-3237]fix connectionMonitor in RestProtocol seems not work #3237 (#3455) * extract method to cache default extension name (#3456) * possibly bug fix (#3460) * Merge pull request #3470, Apache parent pom version is updated to 21. * A follow up of 6058846, update apache pom version to 21 * Class comment error (#3481) * enhancement: extract duplicated method calls to variable (#3482) extract duplicated method calls to variable * [Enhancement]: language level migration (#3485) * use java7 diamond operator * replace Collections.sort with List.sort * extract duplicated code blocks * use StandardCharsets.UTF_8 * use try-with-resources * use java7 diamond operator * enhance log message * fix unit tests failures * Branch refactor version (#3471) * fix typo (#3491) * [Enhancement]: RestProtocol (#3480) * Merge pull request #3466, Condition is not properly used. fixes #1917 * fix issue#1293: A question for ReferenceConfigCache. (#3505) * [Enhancement]: refactor categorizing with Collectors.groupingBy (#3490) * Fix invocation of toString on an array (#3507) Fix invocation of toString on an array * Fix inefficient use of keySet iterator instead of entrySet iterator (#3508) Fix inefficient use of keySet iterator instead of entrySet iterator * Boolean class use method toString() instead of String.valueof() (#3495) * Merge pull request #3515, add metadataFactory SPI config for all-in-one shade jar. Fixes #3514, missing redis metadata SPI extension in dubbo.jar. * Merge pull request #3513, bump up hessian-lite version Fixes #3423. * [Dubbo-3106]Make getRegistered return unmodifiable collection. #3106 (#3425) * make getRegistered return unmodifiable collection. #3106 * fix ci failure * Merge pull request #3527 Bricks-Man/incubator-dubbo, fix accidentally check exchanger in setDispatcher fixes #3518 * Fix UT error following PR #3527 * Cache CompiledScript #390 (#3524) * Merge pull request #3532 from beiwei30:use-concurrent-map, avoid using synchronized. * Fix context filter's bug (#3526) * Modify MetadataReportRetry ThreadName (#3550) * fix: rename the thread name from DubboRegistryFailedRetryTimer to DubboMetadataReportRetryTimer in MetadataReportRetry * fix issue #3533 (#3548) * fix issue #195: @reference check=false不生效 (#3530) * Merge pull request #3528, fixes #208, setOnreturn does not work with generic invocation. * fix issue #274: monitor的cluster一定是failsafe的，而且无法修改 (#3523) * fix issue #274: monitor的cluster一定是failsafe的，而且无法修改 * remove unused import * Merge pull request #3520, fix #538 polish the process of deciding the ip to bind. * Fix npe when package is null. (#3557) * a more elegant way (#3567) * Correct security report link. * Add go implementation link. * fix DubboCodec re-implements Codec2 #2977 (#3547) * Merge pull request #3566, optimize compareTo of Router to guarantee consistent behaviour. * Merge pull request #3577, rmi support generic. Fixes #2779 * Fixes #3367, fail to parse config text with white space (#3589) * [DUBBO-3476]: NullPointerException happened when using SpringContainer.getContext() (#3600) * #3476: NullPointerException happened when using SpringContainer.getContext() * rollback change for DEFAULT_SPRING_CONFIG * replace magic string "dubbo" with constants (#3602) * dubbo-parent是头文件不包含依赖。替换成dubbo, 同时增加zk连接的依赖。因为curator-framework在dubbo-dependencies当中去除掉了zookeeper的包，所以外部使用的时候需要额外配置. (#3516) * Merge pul request #3607, introduce dubbo-dependencies-zookeeper. * Merge RestProtocolTest.java (#3597) * [DUBBO-3494]: Refactor URL to URLBuilder (#3500) * refactor URL to URLBuilder. #3494 * remove unrelated changes * replace more with URLBuilder * fix ci failure * remove unnecessary comment * Dubbo-3473 Fix Not Properly Closed Resources (#3474) * Dump TagRouterRule (#3536) Dump TagRouterRule since the TagRouterRule can be changed to `null` by ConfigCenter * Merge pull request #3578, fixes #3289, enhance tagRoute: support ip expression match. * Merge pull request #2614, follow up for issue #195. * [Dubbo-3367] Fail to parse config text with white space (#3590) * Merge pull request #3558, check if remoteGroup is empty or not. Fixes #3555. * Fluent style builder API support(#3431) (#3549) * Update maven central badge. * Merge pull request #3593, Consul support for Registry and Metadata. * [Dubbo-808] Support etcd registry (#3605) * Merge https://github.com/dubbo/dubbo-registry-etcd into incubator-dubbo * Add UT to ConfigurationUtilsTest * rename dubbo-ops to dubbo-admin (#3628) * typo for AccessLogFilter (#3633) * Fix some etcd3 registry bugs. (#3632) * fix some bugs. * fix typo * cancel keep alive if recovery failed. * remove duplicate license header. * [Dubbo-3570] repackage compatible enhancement. (#3622) * Fixes #3570, NoSuchMethodError are thrown when add custorm Filter using dubbo2.6.5 and JDK1.6 and upgrade to dubbo2.7.0 * Add compatible UT * fix UT * Replace RpcStatus to count (#2984) (#3636) * Fix when qos is disable,log will print every time. (#3397) * fix when qos is disable,log will print every time. * change qos server boos thread number 1 * add openjdk to travis (#3300) * add openjdk to travis * add openjdk to travis * Merge pull request #3647, workaround to fix #3646. * add metrics integration #3598 (#3643) * add metrics integration * add license * Merge pull request #3639, Add equivalent annotation support for MethodConfig. Fixes #2045 * Fixes #3478, #3477 and #3445 * fix heartbeat internal (#3579) * Merge pull request #3603, configcenter share zookeeper connection with registry. Fixes #3288 * correct spelling error (#3645) * make snakeyaml transitive, governance rule relies on this dependency to work. (#3659) * check null for path before call rest server (#3665) * [Dubbo-3653] etcd as config center (#3663) * Minor refactor, no functinoal change. * Separate ConnectionStateListener * Simplify code * Fix typo * Support get external config from etcd config center * Polish diamond operator * Initial etcd support as config center * Add a put interface for JEtcdClient * Enhanced Etcd config center support with the ability to watch and cancel watch * Polish code * Distinguish modification event and delete event * Add etcd registry and configcenter to dubbo-all * Watch again when connection is re-established * Polish code and fix some documentation errors (#3655) * [Dubbo-3657] Fix junit test failed (#3658) * Improve the checking of lease id. #3684 (#3692) It looks good. * Optimize DefaultTpsLimiter (#3654) * Correct @parameter config of field of ConfigCenterConfig (#3688) * fix-3678 (#3681) * Add unit test for unpack and stick pack of dubbo and telent (#3703) * fix compile error after merged master branch * remove useless imports * add AddressListener into RegistryDirectory

khanimteyaz and others added 9 commits December 7, 2018 00:57

Merge pull request #1 from apache/master

4704218

Update from upstream

Merge pull request #3 from apache/master

3c972a1

master synch

Merge pull request #4 from apache/master

6eaf970

Sync of master

Merge pull request #6 from apache/master

61dfa3c

Master sync

Merge pull request #7 from apache/master

565aa1a

Master sync

Merge pull request #8 from apache/master

5b67c0f

Master sync

Merge pull request #9 from apache/master

a868472

Master sync

Merge pull request #10 from apache/master

ad7c7cf

master sync

apache#3026 Access log related changed

6cb044c

kun-song reviewed Jan 19, 2019

View reviewed changes

dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/filter/AccessLogFilter.java Show resolved Hide resolved

kun-song reviewed Jan 19, 2019

View reviewed changes

dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/filter/AccessLogFilter.java Outdated Show resolved Hide resolved

kun-song requested changes Jan 19, 2019

View reviewed changes

dubbo-common/src/main/java/org/apache/dubbo/common/utils/DateUtil.java Outdated Show resolved Hide resolved

dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/filter/AccessLogFilter.java Outdated Show resolved Hide resolved

Reviwe comment incorporated given by @satansk and removed unuded method

bf6e9ab

ralf0131 added the level/medium label Jan 20, 2019

beiwei30 requested changes Jan 21, 2019

View reviewed changes

dubbo-common/pom.xml Outdated Show resolved Hide resolved

khanimteyaz added 3 commits January 21, 2019 22:18

Incorporated @beiwei30 review comment, incorporated common-lang3 time…

9a98de6

… package modified version

Merge branch 'acesslog-dateformat-enhancemnet' of https://github.com/…

5224d58

…khanimteyaz/incubator-dubbo into acesslog-dateformat-enhancemnet

Added rat entry for common lang3 FastDateFormat related java files

896066d

beiwei30 added 2 commits January 31, 2019 16:49

Merge branch 'acesslog-dateformat-enhancemnet' of https://github.com/…

d1342dd

…khanimteyaz/incubator-dubbo into khanimteyaz-acesslog-dateformat-enhancemnet

switch back to jdk's dateformatter

52b3353

Merge pull request #11 from beiwei30/khanimteyaz-acesslog-dateformat-…

4f6d008

…enhancemnet switch to use jdk's dateformatter

beiwei30 approved these changes Feb 2, 2019

View reviewed changes

beiwei30 merged commit 5146f6d into apache:master Feb 2, 2019

dubbo-bot mentioned this pull request Feb 8, 2019

[WeeklyReport] Weekly report for Dubbo 2019-2-1 to 2019-2-8 #3439

Closed

beiwei30 added this to the 2.7.1 milestone Mar 4, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Acesslog dateformat enhancemnet #3274

Acesslog dateformat enhancemnet #3274

khanimteyaz commented Jan 18, 2019

codecov-io commented Jan 18, 2019 •

edited

Loading

khanimteyaz commented Jan 22, 2019

carryxyh commented Jan 22, 2019

beiwei30 commented Jan 22, 2019

khanimteyaz commented Jan 22, 2019 •

edited

Loading

khanimteyaz commented Jan 27, 2019

tswstarplanet commented Jan 28, 2019

kun-song commented Jan 28, 2019

beiwei30 commented Jan 31, 2019

beiwei30 commented Jan 31, 2019

khanimteyaz commented Feb 1, 2019

khanimteyaz commented Feb 1, 2019

beiwei30 commented Feb 2, 2019

khanimteyaz commented Feb 2, 2019

Acesslog dateformat enhancemnet #3274

Acesslog dateformat enhancemnet #3274

Conversation

khanimteyaz commented Jan 18, 2019

What is the purpose of the change

Brief changelog

Verifying this change

codecov-io commented Jan 18, 2019 • edited Loading

Codecov Report

khanimteyaz commented Jan 22, 2019

carryxyh commented Jan 22, 2019

beiwei30 commented Jan 22, 2019

khanimteyaz commented Jan 22, 2019 • edited Loading

khanimteyaz commented Jan 27, 2019

tswstarplanet commented Jan 28, 2019

kun-song commented Jan 28, 2019

beiwei30 commented Jan 31, 2019

beiwei30 commented Jan 31, 2019

khanimteyaz commented Feb 1, 2019

khanimteyaz commented Feb 1, 2019

beiwei30 commented Feb 2, 2019

khanimteyaz commented Feb 2, 2019

codecov-io commented Jan 18, 2019 •

edited

Loading

khanimteyaz commented Jan 22, 2019 •

edited

Loading