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.

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

feat(be): implement feature of judging in java #2323

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

gyunseo
Copy link
Contributor

@gyunseo gyunseo commented Jan 22, 2025

Description

Additional context


Before submitting the PR, please make sure you do the following

Demo 방법

도커(docker file을 수정했습니다)를 새로 빌드하고,

cd apps/iris/
sudo -E go run main.go

http://localhost:15672/#/queues/vh/client.q.judge.submission로 접속하기.

properties
message_id: '1'
type: judge

payload:

{
  "problemId": 1,
  "code": "import java.util.Scanner;\n\npublic class Main {\n    public static void main(String[] args) {\n        Scanner sc = new Scanner(System.in);\n        int a = sc.nextInt();\n        int b = sc.nextInt();\n        System.out.println(a + b);\n    }\n}",
  "language": "Java",
  "timeLimit": 1000,
  "memoryLimit": 268435456
}

publish하고 irisi에서 제대로 채점되는지 watch하면 됩니다.

@coolify-skkuding
Copy link

coolify-skkuding bot commented Jan 22, 2025

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2025-02-10 07:32:11 CET

@gyunseo gyunseo changed the title feat(be)1158 judge in java feat(be): implement feature of judging in java Jan 22, 2025
@gyunseo gyunseo self-assigned this Jan 22, 2025
@gyunseo gyunseo requested a review from Jaehyeon1020 January 22, 2025 11:39
@gyunseo gyunseo requested a review from jspark2000 February 10, 2025 07:23
@gyunseo
Copy link
Contributor Author

gyunseo commented Feb 10, 2025

기존에 MemoryLimitCheckOnly가 제대로 적용이 안되던 걸 고쳤습니다.

@@ -123,7 +123,7 @@ func NewLangConfig(file file.FileManager, javaPolicyPath string) *langConfig {
CompileArgs: "{srcPath} -d {exeDir} -encoding UTF8",
RunCommand: "/usr/bin/java",
RunArgs: "-cp {exeDir} " +
"-XX:MaxRAM={maxMemory}k " +
Copy link
Contributor Author

@gyunseo gyunseo Feb 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jspark2000 이 당시에 뒤에 k를 붙였던 이유가 있었나요? 막연히 kilobyte라고 생각해서 일단 떼어 냈습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant