Details
-
Task
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
-
Docs Required
Description
Problem
At the moment, the org.apache.ignite.internal.storage.pagememory.configuration.schema.PageMemoryStorageEngineConfigurationSchema contains configuration for in-memory and persistent org.apache.ignite.internal.pagememory.configuration.schema.PageMemoryDataRegionConfigurationSchema, which can be inconvenient for the user for several reasons:
- PageMemoryDataRegionConfigurationSchema contains the configuration for in-memory and the persistent case, which can be confusing because it's not obvious which properties to set for each;
- User does not have the ability to set a different size PageMemoryStorageEngineConfigurationSchema#pageSize for in-memory and the persistent case;
- When creating a table through SQL, it would be more convenient for the user to simply specify the engine and use the default region than specify the data region, let's look at the examples.
CREATE TABLE user (id INT PRIMARY KEY, name VARCHAR(255)) ENGINE pagememory dataRegion='in-memory' CREATE TABLE user (id INT PRIMARY KEY, name VARCHAR(255)) ENGINE pagememory dataRegion='persistnet'
CREATE TABLE user (id INT PRIMARY KEY, name VARCHAR(255)) ENGINE in-memory-pagememory CREATE TABLE user (id INT PRIMARY KEY, name VARCHAR(255)) ENGINE persistnet-pagememory
Implementation proposal
Divide by two (in-memory and persistent):
- org.apache.ignite.internal.pagememory.configuration.schema.PageMemoryDataRegionConfigurationSchema
- org.apache.ignite.internal.storage.pagememory.configuration.schema.PageMemoryStorageEngineConfigurationSchema
- org.apache.ignite.internal.storage.pagememory.PageMemoryStorageEngine
Attachments
Issue Links
- blocks
-
IGNITE-17208 Change storage engine names based on PageMemory
- Resolved
-
IGNITE-17210 Modification of documentation on data regions for PageMemory
- Resolved
- Dependency
-
IGNITE-17197 Change the default storage engine (for tables)
- Resolved
- is blocked by
-
IGNITE-17148 Support for abstract configuration
- Resolved
- links to