-
Notifications
You must be signed in to change notification settings - Fork 242
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.
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
fix(amazonq): for /test internal users project payload limit increased to 1.5GB #5322
Conversation
...ty/src/software/aws/toolkits/jetbrains/services/codewhisperer/util/CodeWhispererConstants.kt
Fixed
Show fixed
Hide fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, if possible increase the payload size to 2GB for testing with internal customers.
@@ -60,7 +63,12 @@ class CodeTestSessionConfig( | |||
*/ | |||
fun overallJobTimeoutInSeconds(): Long = DEFAULT_CODE_SCAN_TIMEOUT_IN_SECONDS | |||
|
|||
fun getPayloadLimitInBytes(): Long = DEFAULT_PAYLOAD_LIMIT_IN_BYTES | |||
fun getPayloadLimitInBytes(): Long = | |||
if (isInternalUser(getStartUrl(project))) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can this limit be fetched from the service instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not right now. This needs service changes which would take time, so we used existing client options. This change is temporary right now for debugging the issue. we will make this permanent across the customers(external or internal) once we have data
Types of changes
Description
for /test internal users project payload limit increased to 1.5GB
Checklist
License
I confirm that my contribution is made under the terms of the Apache 2.0 license.