You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2021/07/31 00:45:57 UTC

[GitHub] [kafka] wcarlson5 opened a new pull request #11153: MINOR: Port fix to other StoreQueryIntegrationTests

wcarlson5 opened a new pull request #11153:
URL: https://github.com/apache/kafka/pull/11153


   https://github.com/apache/kafka/pull/11129 fixed one of the tests, This ports the fix to the others as well
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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: jira-unsubscribe@kafka.apache.org

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



[GitHub] [kafka] wcarlson5 edited a comment on pull request #11153: MINOR: Port fix to other StoreQueryIntegrationTests

Posted by GitBox <gi...@apache.org>.
wcarlson5 edited a comment on pull request #11153:
URL: https://github.com/apache/kafka/pull/11153#issuecomment-893137087


   @vvcephei looks like it is working now
   
   EDIT: nvm odd failure
   
   `java.lang.AssertionError: Unexpected exception thrown while getting the value from store.
   Expected: is one of {<a string containing "Cannot get state store source-table because the stream thread is PARTITIONS_ASSIGNED, not RUNNING">, <a string containing "The state store, source-table, may have migrated to another instance">}
        but: was "Cannot get state store source-table because the stream thread is PARTITIONS_ASSIGNED, not RUNNING"
   `


-- 
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: jira-unsubscribe@kafka.apache.org

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



[GitHub] [kafka] wcarlson5 commented on pull request #11153: MINOR: Port fix to other StoreQueryIntegrationTests

Posted by GitBox <gi...@apache.org>.
wcarlson5 commented on pull request #11153:
URL: https://github.com/apache/kafka/pull/11153#issuecomment-890266944


   @vvcephei @patrickstuedi  @ableegoldman Can I get a +1 to use the same fix for the other tests in this file?


-- 
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: jira-unsubscribe@kafka.apache.org

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



[GitHub] [kafka] vvcephei merged pull request #11153: MINOR: Port fix to other StoreQueryIntegrationTests

Posted by GitBox <gi...@apache.org>.
vvcephei merged pull request #11153:
URL: https://github.com/apache/kafka/pull/11153


   


-- 
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: jira-unsubscribe@kafka.apache.org

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



[GitHub] [kafka] wcarlson5 commented on pull request #11153: MINOR: Port fix to other StoreQueryIntegrationTests

Posted by GitBox <gi...@apache.org>.
wcarlson5 commented on pull request #11153:
URL: https://github.com/apache/kafka/pull/11153#issuecomment-893137087


   @vvcephei looks like it is working now


-- 
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: jira-unsubscribe@kafka.apache.org

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



[GitHub] [kafka] wcarlson5 edited a comment on pull request #11153: MINOR: Port fix to other StoreQueryIntegrationTests

Posted by GitBox <gi...@apache.org>.
wcarlson5 edited a comment on pull request #11153:
URL: https://github.com/apache/kafka/pull/11153#issuecomment-893137087


   @vvcephei looks like it is working now
   
   EDIT: nvm odd failure


-- 
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: jira-unsubscribe@kafka.apache.org

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



[GitHub] [kafka] wcarlson5 edited a comment on pull request #11153: MINOR: Port fix to other StoreQueryIntegrationTests

Posted by GitBox <gi...@apache.org>.
wcarlson5 edited a comment on pull request #11153:
URL: https://github.com/apache/kafka/pull/11153#issuecomment-893137087


   @vvcephei looks like it is working now
   
   EDIT: nvm odd failure, I don't think that should be a failure. Trying to reproduce locally
   
   `java.lang.AssertionError: Unexpected exception thrown while getting the value from store.
   Expected: is one of {<a string containing "Cannot get state store source-table because the stream thread is PARTITIONS_ASSIGNED, not RUNNING">, <a string containing "The state store, source-table, may have migrated to another instance">}
        but: was "Cannot get state store source-table because the stream thread is PARTITIONS_ASSIGNED, not RUNNING"
   `


-- 
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: jira-unsubscribe@kafka.apache.org

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



[GitHub] [kafka] wcarlson5 commented on pull request #11153: MINOR: Port fix to other StoreQueryIntegrationTests

Posted by GitBox <gi...@apache.org>.
wcarlson5 commented on pull request #11153:
URL: https://github.com/apache/kafka/pull/11153#issuecomment-893137087


   @vvcephei looks like it is working now


-- 
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: jira-unsubscribe@kafka.apache.org

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