You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@samza.apache.org by GitBox <gi...@apache.org> on 2022/01/21 00:03:33 UTC

[GitHub] [samza] ajothomas opened a new pull request #1577: SAMZA-2718: TaskContext.getTable should not upcast ReadWriteUpdateTable to ReadWriteTable

ajothomas opened a new pull request #1577:
URL: https://github.com/apache/samza/pull/1577


   **Issue:**
   TaskContext.getTable currently upcasts ReadWriteUpdateTable to ReadWriteTable. This is incorrect as update(), updateAsync() calls to the returned table (ReadWriteUpdateTable) will work when it is not intended to work.
   
   **Changes:**
   Create a delegate class implementing ReadWriteTable which is composed of ReadWriteUpdateTable. All method calls are delegated to ReadWriteUpdateTable's methods except for update methods.
   
   **Tests**
   ./gradlew build
   


-- 
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 comment.

To unsubscribe, e-mail: commits-unsubscribe@samza.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [samza] xinyuiscool merged pull request #1577: SAMZA-2718: TaskContext.getTable should not upcast ReadWriteUpdateTable to ReadWriteTable

Posted by GitBox <gi...@apache.org>.

xinyuiscool merged pull request #1577:
URL: https://github.com/apache/samza/pull/1577


   


-- 
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 comment.

To unsubscribe, e-mail: commits-unsubscribe@samza.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org