Skip to content

Assignment 1: Completed #974

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

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

Conversation

siddham04
Copy link

  1. Grammar Modifications:
    Added lexer tokens BYTE_SIZE and TIME_DURATION to the grammar (Directives.g4).

2.API Updates:
Created new Java classes ByteSize and TimeDuration extending Token, which handle parsing and conversion of byte size and time duration strings.

3.Core Parser Updates:
Implemented visitor methods (visitByteSizeArg and visitTimeDurationArg) to handle parsing of byte size and time duration values.

4.New Directive Implementation:
Created a new aggregate directive, aggregate-stats, which computes the sum or average of byte size and time duration columns.

5.Testing:
Added unit tests for ByteSize and TimeDuration classes to ensure correct parsing and value retrieval.

Checklist:

  • Grammar changes (lexer and parser) for byte size and time duration.
  • API updates for ByteSize and TimeDuration classes.
  • New aggregate-stats directive implementation.
  • Unit and parser tests for the new features.
  • Successful build and test execution.

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