Skip to content
This repository has been archived by the owner on May 23, 2023. It is now read-only.

GraphRecordParser class is missing #194

Closed
dgiesbrecht opened this issue Apr 16, 2015 · 5 comments
Closed

GraphRecordParser class is missing #194

dgiesbrecht opened this issue Apr 16, 2015 · 5 comments

Comments

@dgiesbrecht
Copy link

Support for several key components of Graph Tiling appear to be missing in the current code (e.g., oculus.binning.graph.edges.type, oculus.binning.graph.edges.type.index fields, etc.).

These, and other Graph Tiling parameters, used to be handled in the com.oculusinfo.tilegen.datasets.GraphRecordParser class, which appears to have been removed.

@cdbethune
Copy link
Member

@nkronenfeld Any chance all of the refactoring done in support of tiling tasks caused this to be removed?

@nkronenfeld
Copy link
Contributor

I'm not finding anything by that name.
Among other things:
git log --all -- '**/GraphRecordParser.scala' shows nothing.
Is there somewhere in the code that refers to it to help track it down?

@dgiesbrecht
Copy link
Author

It used to be in com.oculusinfo.tilegen.datasets.GraphRecordParser
Specifically in RecordParser.scala

On 2015-04-21 9:50 PM, Nathan Kronenfeld wrote:

I'm not finding anything by that name.
Among other things:
git log --all -- '**/GraphRecordParser.scala' shows nothing.
Is there somewhere in the code that refers to it to help track it down?


Reply to this email directly or view it on GitHub
#194 (comment).

@nkronenfeld
Copy link
Contributor

ah, not a file, just a class. Ok, I'll take a further look

@nkronenfeld
Copy link
Contributor

This was removed in the move from CSV to SchemaRDD/DataFrame-based tiling. It's probably better replaced with a filter pipeline op now.

@dschroh dschroh closed this as completed Apr 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants