Re: [PR] TIKA-4260 -- add ParseContext to fetchers and emitters [tika]

2024-06-03 Thread via GitHub


tballison closed pull request #1776: TIKA-4260 -- add ParseContext to fetchers 
and emitters
URL: https://github.com/apache/tika/pull/1776


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tika.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] TIKA-4260 -- add ParseContext to fetchers and emitters [tika]

2024-06-03 Thread via GitHub


tballison commented on PR #1776:
URL: https://github.com/apache/tika/pull/1776#issuecomment-2144945579

   I merged this into @nddipiazza 's TIKA-4252 PR.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tika.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] TIKA-4260 -- add ParseContext to fetchers and emitters [tika]

2024-05-24 Thread via GitHub


tballison commented on PR #1776:
URL: https://github.com/apache/tika/pull/1776#issuecomment-2130252325

   I'm now getting a clean build with `-DskipTests` lol... That's a step at 
least.
   
   The big TODO is to add serialization of the ParseContext in 
https://github.com/apache/tika/blob/main/tika-serialization/src/main/java/org/apache/tika/metadata/serialization/JsonFetchEmitTuple.java


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tika.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] TIKA-4260 -- add ParseContext to fetchers and emitters [tika]

2024-05-24 Thread via GitHub


tballison commented on PR #1776:
URL: https://github.com/apache/tika/pull/1776#issuecomment-2129532368

   Current status -- only working in tika-core. Much more needs to be done 
throughout the repo to get this working.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tika.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] TIKA-4260 -- add ParseContext to fetchers and emitters [tika]

2024-05-24 Thread via GitHub


tballison opened a new pull request, #1776:
URL: https://github.com/apache/tika/pull/1776

   
   
   Thanks for your contribution to [Apache Tika](https://tika.apache.org/)! 
Your help is appreciated!
   
   Before opening the pull request, please verify that
   * there is an open issue on the [Tika issue 
tracker](https://issues.apache.org/jira/projects/TIKA) which describes the 
problem or the improvement. We cannot accept pull requests without an issue 
because the change wouldn't be listed in the release notes.
   * the issue ID (`TIKA-`)
 - is referenced in the title of the pull request
 - and placed in front of your commit messages surrounded by square 
brackets (`[TIKA-] Issue or pull request title`)
   * commits are squashed into a single one (or few commits for larger changes)
   * Tika is successfully built and unit tests pass by running `mvn clean test`
   * there should be no conflicts when merging the pull request branch into the 
*recent* `main` branch. If there are conflicts, please try to rebase the pull 
request branch on top of a freshly pulled `main` branch
   * if you add new module that downstream users will depend upon add it to 
relevant group in `tika-bom/pom.xml`.
   
   We will be able to faster integrate your pull request if these conditions 
are met. If you have any questions how to fix your problem or about using Tika 
in general, please sign up for the [Tika mailing 
list](http://tika.apache.org/mail-lists.html). Thanks!
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tika.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org