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

3.7.0 disable isNil optimization in seaside extents #331

Open
dalehenrich opened this issue Aug 18, 2023 · 0 comments
Open

3.7.0 disable isNil optimization in seaside extents #331

dalehenrich opened this issue Aug 18, 2023 · 0 comments

Comments

@dalehenrich
Copy link
Member

dalehenrich commented Aug 18, 2023

In a standard 3.7.0 image, the isNil selector was optimized (i.e., not sent) the adverse effect on the GLASS/GsDevKit ecosystem wasn't noticed until too late in the release cycle to properly address the issues, so the decision was made to turn off isNil optimization in seaside extents for the time being ... Visit this link for more details about re-enabling isNil optimizations in GsDevKit_home extents.

Changes need to be made to the GLASS bootstrap process to disable isNil optimization and an additional option will be added to upgradeSeasideImage script to leave optimization enabled, if you know or want to find out if it is necessary to disable isNil optimization or not in your application ---

The known issues are:

  1. unit test failures after bootstrapping glass into extent0.dbf (i.e., running $GS_HOME/bin/bootstrapGLASS)
  2. webSocket failures in a GsDevKit_home image, with the optional webSocket group loaded

However, if your application happens to rely on installing a custom isNil method or overriding an existing isNil method, then you must disable isNil optimization until you've changed your application.

Work is planned to continue after 3.7.0 is release to address the above issues.

dalehenrich added a commit that referenced this issue Aug 18, 2023
dalehenrich added a commit that referenced this issue Aug 18, 2023
dalehenrich added a commit that referenced this issue Aug 18, 2023
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

No branches or pull requests

1 participant