diff --git a/polaris-core/src/main/java/org/apache/polaris/core/persistence/AtomicOperationMetaStoreManager.java b/polaris-core/src/main/java/org/apache/polaris/core/persistence/AtomicOperationMetaStoreManager.java index 9c35a7805a..5e598f9b47 100644 --- a/polaris-core/src/main/java/org/apache/polaris/core/persistence/AtomicOperationMetaStoreManager.java +++ b/polaris-core/src/main/java/org/apache/polaris/core/persistence/AtomicOperationMetaStoreManager.java @@ -1226,7 +1226,8 @@ private void revokeGrantRecord( properties.put( PolarisTaskConstants.TASK_TYPE, String.valueOf(AsyncTaskType.ENTITY_CLEANUP_SCHEDULER.typeCode())); - properties.put("data", PolarisObjectMapperUtil.serialize(refreshEntityToDrop)); + properties.put( + PolarisTaskConstants.TASK_DATA, PolarisObjectMapperUtil.serialize(refreshEntityToDrop)); PolarisBaseEntity.Builder taskEntityBuilder = new PolarisBaseEntity.Builder() .properties(PolarisObjectMapperUtil.serializeProperties(properties)) diff --git a/polaris-core/src/main/java/org/apache/polaris/core/persistence/transactional/TransactionalMetaStoreManagerImpl.java b/polaris-core/src/main/java/org/apache/polaris/core/persistence/transactional/TransactionalMetaStoreManagerImpl.java index 34231cfda5..50ebc52a78 100644 --- a/polaris-core/src/main/java/org/apache/polaris/core/persistence/transactional/TransactionalMetaStoreManagerImpl.java +++ b/polaris-core/src/main/java/org/apache/polaris/core/persistence/transactional/TransactionalMetaStoreManagerImpl.java @@ -1428,7 +1428,8 @@ private void bootstrapPolarisService( properties.put( PolarisTaskConstants.TASK_TYPE, String.valueOf(AsyncTaskType.ENTITY_CLEANUP_SCHEDULER.typeCode())); - properties.put("data", PolarisObjectMapperUtil.serialize(refreshEntityToDrop)); + properties.put( + PolarisTaskConstants.TASK_DATA, PolarisObjectMapperUtil.serialize(refreshEntityToDrop)); PolarisBaseEntity.Builder taskEntityBuilder = new PolarisBaseEntity.Builder() .id(ms.generateNewIdInCurrentTxn(callCtx)) diff --git a/polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/BasePolarisMetaStoreManagerTest.java b/polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/BasePolarisMetaStoreManagerTest.java index b8db196122..12096be46a 100644 --- a/polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/BasePolarisMetaStoreManagerTest.java +++ b/polaris-core/src/testFixtures/java/org/apache/polaris/core/persistence/BasePolarisMetaStoreManagerTest.java @@ -41,6 +41,7 @@ import org.apache.polaris.core.entity.PolarisEntity; import org.apache.polaris.core.entity.PolarisEntitySubType; import org.apache.polaris.core.entity.PolarisEntityType; +import org.apache.polaris.core.entity.PolarisTaskConstants; import org.apache.polaris.core.entity.TaskEntity; import org.apache.polaris.core.persistence.pagination.PageToken; import org.assertj.core.api.Assertions; @@ -293,8 +294,8 @@ protected void testLoadTasks() { .extracting( e -> PolarisObjectMapperUtil.deserializeProperties(e.getProperties())) .asInstanceOf(InstanceOfAssertFactories.map(String.class, String.class)) - .containsEntry("lastAttemptExecutorId", executorId) - .containsEntry("attemptCount", "1")); + .containsEntry(PolarisTaskConstants.LAST_ATTEMPT_EXECUTOR_ID, executorId) + .containsEntry(PolarisTaskConstants.ATTEMPT_COUNT, "1")); Set firstTasks = taskList.stream().map(PolarisBaseEntity::getName).collect(Collectors.toSet());