Skip to content

jvm缓存#229

Open
xiaoxude wants to merge 14 commits intonetease-lcap:mainfrom
xiaoxude:feat/sort-map
Open

jvm缓存#229
xiaoxude wants to merge 14 commits intonetease-lcap:mainfrom
xiaoxude:feat/sort-map

Conversation

@xiaoxude
Copy link
Collaborator

@xiaoxude xiaoxude commented Sep 8, 2024

jvm缓存#206

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2024

关联ACTION

jvm-cache/存在错误:

不存在合法的package配置,当前目录下找不到pom.xml或package.json

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2024

关联ACTION

jvm-cache/存在错误:

不存在合法的package配置,当前目录下找不到pom.xml或package.json

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2024

关联ACTION

jvm-cache存在错误:

当前package缺乏必要文件:
 x 缺少依赖库使用文档说明.docx


资产共建大赛提交内容中原有的截图,现升级为依赖库使用说明文档。现交内容包括
1. 依赖库代码本身;2. readme代码说明文档;3. 依赖库使用文档说明。
其中readme面向开发者,使用文档面向使用者。
请自己阅读提交作品路径说明。
https://github.com/netease-lcap/CodeWaveAssetCompetition2024/blob/main/%E8%B5%84%E4%BA%A7%E5%85%B1%E5%BB%BA%E5%A4%A7%E8%B5%9B%E6%8F%90%E4%BA%A4%E5%86%85%E5%AE%B9%E8%AF%B4%E6%98%8E.md

参考作品https://github.com/netease-lcap/CodeWaveAssetCompetition2024/tree/main/demo_bjz_JiazhenBao

tips:README.md和依赖库使用文档说明.docx需要保持文件名称不动。后续我们会对提交的PR自动校验目录结构~

@xiaoxude xiaoxude changed the title Feat/sort map jvm缓存 Sep 8, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2024

关联ACTION

jvm-cache存在错误:

Command failed: mvn clean package -DskipTests

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2024

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

函数命名建议改为getOrComputeAndSet

* @param expireAfterWriteSeconds 过期时间(秒)
*/
@NaslLogic
public String setCache(String key, String value, Long expireAfterWriteSeconds) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

建议对入参增加校验

@github-actions
Copy link
Contributor

@lyjamare
Copy link
Collaborator

功能及代码已验收,待评分

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments