Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature-request] ScriptEnvironmentImplでのスクリプトキャッシュの容量を設定として指定したい #15

Closed
nobeans opened this issue Aug 29, 2019 · 3 comments · Fixed by #17
Assignees
Milestone

Comments

@nobeans
Copy link

nobeans commented Aug 29, 2019

スクリプト数が128を超える場合にキャッシュを有効に利用できるように、任意の数字を指定できると嬉しいです。

https://github.com/seasarorg/mayaa/blob/master/src-impl/org/seasar/mayaa/impl/cycle/script/rhino/ScriptEnvironmentImpl.java#L52

@mitonize
Copy link
Collaborator

org.seasar.mayaa.provider.ServiceProvider にて cacheSize というパラメータ名で指定できるようにするのはいかがでしょうか。
デフォルトは128のままとし、追加で配置する org.seasar.mayaa.provider.ServiceProviderにて指定するイメージです。

org.seasar.mayaa.provider.ServiceProvider

<scriptEnvironment class="org.seasar.mayaa.impl.cycle.script.rhino.ScriptEnvironmentImpl">
    <scope class="org.seasar.mayaa.impl.cycle.scope.ParamScope"/>
    <scope class="org.seasar.mayaa.impl.cycle.scope.HeaderScope"/>
    <scope class="org.seasar.mayaa.impl.cycle.scope.BindingScope"/>

    <parameter name="wrapFactory" value="org.seasar.mayaa.impl.cycle.script.rhino.WrapFactoryImpl"/>
    <parameter name="cacheSize" value="128"/>  <!-- キャッシュのサイズを指定 -->
</scriptEnvironment>

@nobeans
Copy link
Author

nobeans commented May 25, 2020

いいですね。利用側で指定可能であればOKなので、ご提案の対処で十分だと思います。

@mitonize
Copy link
Collaborator

ご確認ありがとうございます。

mitonize added a commit that referenced this issue Jun 3, 2020
#15 ScriptEnvironment内のキャッシュサイズをファクトリ定義XMLで指定する
mitonize added a commit that referenced this issue Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants