[go: nahoru, domu]
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ITChatSessionIntegrationTest failed with IllegalStateException (flaky) #10984

Open
zhumin8 opened this issue Jun 25, 2024 · 0 comments
Open

ITChatSessionIntegrationTest failed with IllegalStateException (flaky) #10984

zhumin8 opened this issue Jun 25, 2024 · 0 comments
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@zhumin8
Copy link
Contributor
zhumin8 commented Jun 25, 2024 •
edited
Loading

ITChatSessionIntegrationTest:sendMessageMixedStreamAndUnary_historyOfFour and ITChatSessionIntegrationTest.sendMessageMixedStreamAndUnary_historyOfFour failed with IllegalStateException
in GraalVM JDK 17 Nightly

Failures (2):
  JUnit Vintage:ITChatSessionIntegrationTest:sendMessageMixedStreamAndUnary_historyOfFour
    MethodSource [className = 'com.google.cloud.vertexai.it.ITChatSessionIntegrationTest', methodName = 'sendMessageMixedStreamAndUnary_historyOfFour', methodParameterTypes = '']
    => java.lang.IllegalStateException: Rerun getHistory() to get cleaned history.
       com.google.cloud.vertexai.generativeai.ChatSession.getHistory(ChatSession.java:357)
       com.google.cloud.vertexai.it.ITChatSessionIntegrationTest.sendMessageMixedStreamAndUnary_historyOfFour(ITChatSessionIntegrationTest.java:120)
       java.base@17.0.8/java.lang.reflect.Method.invoke([Method.java:568](https://cs.corp.google.com/#search/&sq=package:%5Epiper$%20project:openjdk21&q=java/lang/reflect/Method.java:568))
     Caused by: java.lang.IllegalStateException: The last round of conversation will not be added to history because response stream did not finish normally. Finish reason is RECITATION.
       com.google.cloud.vertexai.generativeai.ChatSession.checkFinishReasonAndEditHistory(ChatSession.java:339)
       com.google.cloud.vertexai.generativeai.ChatSession.lambda$checkLastResponseAndEditHistory$0(ChatSession.java:308)
       java.base@17.0.8/java.util.Optional.ifPresent([Optional.java:178](https://cs.corp.google.com/#search/&sq=package:%5Epiper$%20project:openjdk21&q=java/util/Optional.java:178))
       com.google.cloud.vertexai.generativeai.ChatSession.checkLastResponseAndEditHistory(ChatSession.java:305)
       com.google.cloud.vertexai.generativeai.ChatSession.getHistory(ChatSession.java:353)
       [...]
  JUnit Vintage:ITChatSessionIntegrationTest:sendMessageWithNewConfigs_historyContainsFullConversation
    MethodSource [className = 'com.google.cloud.vertexai.it.ITChatSessionIntegrationTest', methodName = 'sendMessageWithNewConfigs_historyContainsFullConversation', methodParameterTypes = '']
    => java.lang.IllegalStateException: The last round of conversation will not be added to history because response stream did not finish normally. Finish reason is RECITATION.
       com.google.cloud.vertexai.generativeai.ChatSession.checkFinishReasonAndEditHistory(ChatSession.java:339)
       com.google.cloud.vertexai.generativeai.ChatSession.lambda$checkLastResponseAndEditHistory$1(ChatSession.java:323)
       java.base@17.0.8/java.util.Optional.ifPresent([Optional.java:178](https://cs.corp.google.com/#search/&sq=package:%5Epiper$%20project:openjdk21&q=java/util/Optional.java:178))
       com.google.cloud.vertexai.generativeai.ChatSession.checkLastResponseAndEditHistory(ChatSession.java:316)
       com.google.cloud.vertexai.generativeai.ChatSession.sendMessage(ChatSession.java:243)
       com.google.cloud.vertexai.generativeai.ChatSession.sendMessage(ChatSession.java:233)

see log for details.
This was resolved in rebuild.

Previously wrongly opened issue in googleapis/sdk-platform-java#2931
@diegomarquezp diegomarquezp added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. priority: p2 Moderately-important priority. Fix may not be included in next release. labels Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zhumin8 @diegomarquezp