Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-23 Thread via GitHub
HeartSaVioR closed pull request #45991: [SPARK-47805][SS] Implementing TTL for MapState URL: https://github.com/apache/spark/pull/45991 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-23 Thread via GitHub
HeartSaVioR commented on PR #45991: URL: https://github.com/apache/spark/pull/45991#issuecomment-2071662156 > [Run / Build modules: pyspark-sql, pyspark-resource, pyspark-testing](https://github.com/ericm-db/spark/actions/runs/8794474954/job/24137701938#logs) > failed 2 hours ago in 26m

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-23 Thread via GitHub
HeartSaVioR commented on PR #45991: URL: https://github.com/apache/spark/pull/45991#issuecomment-2071662357 Thanks! Merging to master. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
HeartSaVioR commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575597023 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575592659 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575586347 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
HeartSaVioR commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575569366 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
HeartSaVioR commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575549218 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575072868 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575071946 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575025178 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575020539 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575011750 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1575010427 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/TTLState.scala: ## @@ -99,6 +98,21 @@ abstract class SingleKeyTTLStateImpl(

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
HeartSaVioR commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1574155684 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-22 Thread via GitHub
HeartSaVioR commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1574155684 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-19 Thread via GitHub
ericm-db commented on PR #45991: URL: https://github.com/apache/spark/pull/45991#issuecomment-2067115942 @HeartSaVioR PTAL, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1571187342 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1571185285 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1571071937 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1571061801 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1571061801 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1571060613 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1570093814 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1570092192 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,308 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1570090219 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/TTLState.scala: ## @@ -99,6 +98,16 @@ abstract class SingleKeyTTLStateImpl(

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1570085488 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1570083230 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1570080402 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-18 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1570079481 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569241785 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,310 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569223183 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,310 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569224035 ## sql/core/src/test/scala/org/apache/spark/sql/streaming/TransformWithMapStateTTLSuite.scala: ## @@ -0,0 +1,310 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569213581 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/ListStateImplWithTTL.scala: ## @@ -137,6 +137,7 @@ class ListStateImplWithTTL[S]( /** Remove

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569214252 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569209877 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/TTLState.scala: ## @@ -59,23 +75,6 @@ trait TTLState { * @return number of values cleaned up.

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569194799 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/TTLState.scala: ## @@ -59,23 +75,6 @@ trait TTLState { * @return number of values cleaned

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569195018 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/TTLState.scala: ## @@ -163,6 +172,115 @@ abstract class SingleKeyTTLStateImpl( } }

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569193194 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StateTypesEncoderUtils.scala: ## @@ -192,6 +195,28 @@ class CompositeKeyStateEncoder[GK, K,

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569192544 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569182582 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569182254 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/ListStateImplWithTTL.scala: ## @@ -137,6 +137,7 @@ class ListStateImplWithTTL[S]( /**

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-17 Thread via GitHub
anishshri-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1569180634 ## sql/api/src/main/scala/org/apache/spark/sql/streaming/StatefulProcessorHandle.scala: ## @@ -108,6 +108,28 @@ private[sql] trait StatefulProcessorHandle extends

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-16 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1567940592 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-16 Thread via GitHub
ericm-db commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1567908966 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-16 Thread via GitHub
sahnib commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1567707784 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [SPARK-47805][SS] Implementing TTL for MapState [spark]

2024-04-16 Thread via GitHub
sahnib commented on code in PR #45991: URL: https://github.com/apache/spark/pull/45991#discussion_r1567707784 ## sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MapStateImplWithTTL.scala: ## @@ -0,0 +1,265 @@ +/* + * Licensed to the Apache Software Foundation